-
Notifications
You must be signed in to change notification settings - Fork 43
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
Entity specific analytics (#1323) #1332
Conversation
* page visit graph * page visit graph * page visit chart * customize chartkick legends
Deployed review-app can be viewed at https://review-1332.violet-test.net |
Deployed review-app can be viewed at https://review-1332.violet-test.net |
@Pralish some feedback:
|
Deployed review-app can be viewed at https://review-1332.violet-test.net |
…shentity_specific_analytics
…ne/violet_rails into Pralishentity_specific_analytics
Deployed review-app can be viewed at https://review-1332.violet-test.net |
@@ -0,0 +1,61 @@ | |||
$(document).on("turbo:load", () => { |
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.
https://nikean.org story platform uses non-turbo navigation but the static site uses turbo. We should accommodate both cases
Deployed review-app can be viewed at https://review-1332.violet-test.net |
@Ayon95 with this analytics PR coming together nicely, we need a way to test it. Could you add analytics instrumentation in the testnet (http://violet-test.net) to the spots marked below? |
Deployed review-app can be viewed at https://review-1332.violet-test.net |
@Pralish Some feedback:
but they are not shown in Analytics V2
For docs, be sure to show an example each for tracking: section views, countable events, watch time, clicks
|
Deployed review-app can be viewed at https://review-1332.violet-test.net |
@ajaystha42 , @donrestarone ,thanks for the feedbacks
That was the design requirement.
@donrestarone, didn't we decide to not implement that? Wouldn't having redirection on count on form submission make the design inconsistent with the rest of the section? |
yup! The current implementation is fine about the analytics mapping you will need to add the permission for it (please call it |
@@ -9,8 +9,9 @@ | |||
-# this is a hack until I figure out how to get chartkick working without importing the whole application js pack | |||
- if controller_name == 'dashboard' | |||
= javascript_pack_tag 'application', 'data-turbo-track': 'reload' | |||
|
|||
|
|||
%script{type: "text/javascript", src: "//cdn.jsdelivr.net/momentjs/latest/moment.min.js"} |
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.
@Pralish can we install these instead of pulling the latest from a CDN? We are vulnerable to breakages due to changes with the current implementation
@Pralish found a bug, the custom events are not scoped by time period (eg: 1 month, 3 months etc): |
Deployed review-app can be viewed at https://review-1332.violet-test.net |
@Pralish I took a look at the latest commits and the Date dropdown is broken on mobile/desktop and there is broken HTML at the bottom of the page: RPReplay_Final1675007003.MP4on desktop the viewport is broken as well: broken.viewport.mov |
Deployed review-app can be viewed at https://review-1332.violet-test.net |
Deployed review-app can be viewed at https://review-1332.violet-test.net |
…ne/violet_rails into Pralishentity_specific_analytics
Deployed review-app can be viewed at https://review-1332.violet-test.net |
RPReplay_Final1675043264.MP4 |
Deployed review-app can be viewed at https://review-1332.violet-test.net |
Addresses: #1292