Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Added Events tracking #21

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Added Events tracking #21

wants to merge 2 commits into from

Conversation

aboritskiy
Copy link

  • added possibility to track more custom system events, like user registration
  • exposed API to add more types and to push events for tracking engine.
  • fixed adminhtml translation
  • little cleanup: removed commented out code

aboritskiy and others added 2 commits August 25, 2016 15:27
- added possibility to track more custom system events, like user
c9fbf75 
   registration
 - exposed API to add more types and to push events for tracking engine.
 - little cleanup: removed commented out code
@aboritskiy
Merge pull request #1 from sitewards/add-events-feature
f308ac6 
Added Events tracking
@aboritskiy
Copy link
Author

the change is breaking in terms that admin user has to go to backend and re-input his conversion settings. Maybe I should have provided migration script.
But would be great to have some feedback before I put more effort into that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@aboritskiy