-
Notifications
You must be signed in to change notification settings - Fork 67
feat: track user events in Countly #282
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.
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
Conversation
@fsdiogo and I just implemented tracking for when a lesson is passed, which is calculated slightly differently for each:
For each of these it excludes any automatic tests we do on page load, so as not to double-count successes caused by us. However, if a user manages to un-pass and then re-pass a lesson (by clicking off of a wrong answer onto a right one and then back again in multiple choice or by resetting and re-submitting code on an exercise), it will be counted as passed again. The event
|
@fsdiogo I realized I had left the code stuck on counting in increments of 20 so I fixed that. :) I also added sample lessons of every type to my |
Just added metrics for when users reset or submit their code (the automatic runs when you enter a previously passed lesson are discarded). |
This rocks! |
We've recently added event tracking for:
Some of these will require further work in the dashboard (hopefully) or manually (hopefully not) to make the data most useful. For example:
tutorialPassed, segmented by tutorial: We've also created a Vue data isTutorialPassed that can be used to update the UI as in the trophy emoji below granted for completion of all lessons in the tutorial (or however we decide to show progress through completed tutorials in the future): |
We're currently blocked by the Countly team, to whom Teri raised this issue today, asking for their help in getting it fixed:
The live site right now is using https://countly.proto.school/sdk/web/countly.min.js but we believe our event tracking won't fully work without that code being updated by their team. |
Countly has updated the server version for us. @fsdiogo on Monday could you please update the source in this branch to https://countly.proto.school/sdk/web/countly.min.js and test that the view tracking works correctly for the linkClick event and that this removes the Cypress error that blocks deployment? |
@terichadbourne I've updated the code to fetch the countly sdk from countly.proto.school and the event tracking works. The tests are still failing, but they are failing in the |
Can you help me understand this fix @fsdiogo? Under what circumstances does window.Cypress exist? Are we making it so we can never test Countly now, if Cypress is always there? |
|
@terichadbourne, can you do a last test run to check if this is ready to go? |
@fsdiogo The segment "tutorial" was actually broken for all events because we merged the PR that changed the word "workshop" to "tutorial" but hadn't made the substitution in the new code. I've fixed that and also updated another counting function to use the Looks to me like we just need to remove the test lessons now, which I can work on. |
Oops, forgot about that one! Cool, take care of that so we can merge this one 💪 |
@fsdiogo Will you please do one last check to confirm I've removed all the test stuff and things are working on your end, then merge when Travis passes? Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀🌔
closes #100
remove before merging:
bb703e0)
WIP

Previously in place:
Heat map (
track_clicks
):Added so far:

Link tracking by href (
track_links
, eventlinkClick
, segmenthref
):Link tracking by linked text (

track_links
, eventlinkClick
, segmenttext
):Scroll map for each page with color coding for what percentage of visitors see the portion of the screen (

track_scrolls
):