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.

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

Merged
merged 43 commits into from
Jan 30, 2023
Merged

Conversation

donrestarone
Copy link
Contributor

@donrestarone donrestarone commented Jan 3, 2023

Addresses: #1292
Screen Shot 2023-01-04 at 8 07 53 AM

* page visit graph

* page visit graph

* page visit chart

* customize chartkick legends
@github-actions
Copy link

github-actions bot commented Jan 3, 2023

Deployed review-app can be viewed at https://review-1332.violet-test.net

@github-actions
Copy link

github-actions bot commented Jan 4, 2023

Deployed review-app can be viewed at https://review-1332.violet-test.net

@donrestarone
Copy link
Contributor Author

@Pralish some feedback:

  1. Rename Visitor Analytics to Analytics

Screen Shot 2023-01-09 at 10 43 19 AM

  1. unify dropdown style between the two dropdowns shown below. It should be clear to the user that they are dropdowns

Screen Shot 2023-01-09 at 10 44 15 AM

@github-actions
Copy link

Deployed review-app can be viewed at https://review-1332.violet-test.net

@github-actions
Copy link

Deployed review-app can be viewed at https://review-1332.violet-test.net

@@ -0,0 +1,61 @@
$(document).on("turbo:load", () => {
Copy link
Contributor Author

@donrestarone donrestarone Jan 18, 2023

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

@github-actions
Copy link

Deployed review-app can be viewed at https://review-1332.violet-test.net

@donrestarone
Copy link
Contributor Author

@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?
Screen Shot 2023-01-20 at 5 54 59 PM

@github-actions
Copy link

Deployed review-app can be viewed at https://review-1332.violet-test.net

@donrestarone
Copy link
Contributor Author

@Pralish Some feedback:

  1. there are events as indicated in Legacy Analytics,

Screen Shot 2023-01-21 at 12 12 22 PM

but they are not shown in Analytics V2

  1. Please show events in a similar UI as Clicks and Watch Time (name it Events) :

Screen Shot 2023-01-21 at 12 12 29 PM

Clicking on each event should lead to `dashboard/events/:event_slug`
  1. Do not show Clicks / Watch Time / Events if none exist, instead render dynamic docs:

Screen Shot 2023-01-21 at 12 13 37 PM

For docs, be sure to show an example each for tracking: section views, countable events, watch time, clicks

  1. API Form submissions are not shown, they should be shown in a similar UI to Clicks and Watch time where each namespace/submission count should lead to the API Namespace #show page

@Ayon95
Copy link
Contributor

Ayon95 commented Jan 22, 2023

I've added the necessary data attributes to the navbar links and car page links.

Car links:
car-link

Navbar links:
navbar

@github-actions
Copy link

Deployed review-app can be viewed at https://review-1332.violet-test.net

@Pralish
Copy link
Contributor

Pralish commented Jan 28, 2023

@ajaystha42 , @donrestarone ,thanks for the feedbacks

Do we need to add the following showing gain/loss percentage?

That was the design requirement.

API Form submissions are shown, but each namespace/submission count isn't leading to the API Namespace #show page

@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?

@donrestarone
Copy link
Contributor Author

@Pralish

@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 allow analytics mapping):

Screen Shot 2023-01-28 at 9 03 29 AM

@@ -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"}
Copy link
Contributor Author

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

@donrestarone
Copy link
Contributor Author

@Pralish found a bug, the custom events are not scoped by time period (eg: 1 month, 3 months etc):

Screen Shot 2023-01-28 at 10 13 06 AM

@github-actions
Copy link

Deployed review-app can be viewed at https://review-1332.violet-test.net

@donrestarone
Copy link
Contributor Author

donrestarone commented Jan 29, 2023

@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.MP4

on desktop the viewport is broken as well:

broken.viewport.mov

@github-actions
Copy link

Deployed review-app can be viewed at https://review-1332.violet-test.net

@donrestarone
Copy link
Contributor Author

@Pralish I back merged master into this branch (70526b5) we should always keep feature branches caught up with master in the later stages of the development to avoid unforeseen issues

@github-actions
Copy link

Deployed review-app can be viewed at https://review-1332.violet-test.net

@github-actions
Copy link

Deployed review-app can be viewed at https://review-1332.violet-test.net

@donrestarone
Copy link
Contributor Author

⚠️ horizontal scrolling on iPhone on the event show page:

RPReplay_Final1675043264.MP4

⚠️ scoping of the events by date range (fix WIP):

@github-actions
Copy link

Deployed review-app can be viewed at https://review-1332.violet-test.net

@Pralish Pralish changed the title [WIP] Entity specific analytics (#1323) Entity specific analytics (#1323) Jan 30, 2023
@Pralish Pralish marked this pull request as ready for review January 30, 2023 04:31
@donrestarone donrestarone changed the base branch from master to rc January 30, 2023 04:36
@donrestarone donrestarone merged commit 3cb3964 into rc Jan 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants