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

Spike: GA4 #194

Open
5 tasks
jillpe opened this issue Jan 21, 2025 · 9 comments
Open
5 tasks

Spike: GA4 #194

jillpe opened this issue Jan 21, 2025 · 9 comments
Labels

Comments

@jillpe
Copy link

jillpe commented Jan 21, 2025

Description of Work and Deliverables

GA4-Compatible Analytics Reporting

Much of the work for Analytics in Hyku is already complete. The Hyrax Interest Group reports that recording of analytics is functional with GA4, but that the work to render that information in the reporting dashboard is what remains. SoftServ will finish the Dashboard work to connect to the GA4 backend updates restore Google Analytics functionality.

This work is estimated at 0.5 sprints. This SOW does not guarantee a fixed deliverable, but the estimate is given in good faith. Should unexpected complications arise, we’ll work with the client to align expectations and prioritize work within the approved allocation.

The approach to deploying this work will depend on an assessment of difficulty and the state of Hyrax and Hyku releases at the time. The update will be made in Hyrax, and from there Hyku will either be updated with the latest Hyrax or will bring in just the GA4 work as a patch to Hyku. Completion will be the integration of GA4-compatible analytics into Hyku contributed to the current stable release version or packaged with the next release.

SOW

TASKS (+ see comments)

Test that GA4 works and take note of issues below:

debug issues:

  • counts aren't different per tenant
  • file download is not being tracked
  • too many emails - all have the same data regardless of user/tenant
  • UserStatCollectionJob dominates everything else => bulkrax doesn't take precendence
  • analytics side bar

$$$$? =>

Image
@jillpe jillpe added the Pitt label Jan 21, 2025
@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Jan 23, 2025

Hyrax code:

Image

needs to be updated to Hyrax.query_service

cc @kirkkwang

Also, we noticed the job doesn't seem to be tenant specific.

UPDATE: this should be resolved. Kirk submitted a PR and we've since discovered that the job is tenant specific

@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Feb 4, 2025

What happened to the analytics side bar? ref https://notch8.slack.com/files/U08652W0N2Y/F08BSCAQLJF/image.png

re: Per LaRita -

Image

update: i found that adding HYRAX_ANALYTICS_REPORTING="true" reveals it however clicking into it causes errors. I wonder if it's supposed to be deprecated/not used:

Image

@ShanaLMoore
Copy link
Contributor

Nic informed us that in pals prod, for tenants that don't have google analytics set it would fall back to a default. perhaps that's why the tenant counts weren't different in earlier testing???

@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Feb 6, 2025

GA4 PR - samvera/hyrax#6815

https://notch8.slack.com/archives/C087FQ5PG22/p1718385893183039

shana
  10:24 AM
GA4 NOTES:
• Make sure we have access to GA.
• Create a property: [palni-palci-staging.notch8.com](http://palni-palci-staging.notch8.com/).
• need value and property id
• fill in env vars using ops staging. don't need json AND json path - pick one. (json path is in 1 password)
• Add ENV vars to github environment settings
• After deploy - come back to GA dashboard. click around the site a lot - will need to wait until monday to see the results
• [analytics.google.com](http://analytics.google.com/)

@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Feb 6, 2025

should this page be populated with analytic data?

Image

Update: yes it should be, kirk implemented a fix for this page and data is showing 🎉

@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Feb 10, 2025

  • add help text that it must that adding the analytics id to settings is not enough.
    • Google Analytics API must also be enabled
    • registered email must have at least one of these roles: Editor, Admin, Property Access Administrator

@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Feb 10, 2025

Update translations: it's confusing/unclear

Metadata View => work views
Views => File views
Downloads => File Downloads

Image

Image

  • file downloads aren't showing up in the user profile but we see the in google analytics dashabord and the analytics reports a day later

@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Feb 10, 2025

we're going to set down the job prioritization to see if fixing the problem fixes the aggressive UserStatCollectionJob

Going forward we could consider switching to good job.

  • If the job gets deprioritized in GoodJob, is there a chance they would still get delayed multiple days?
  • If we do switch PALs’ jobs backend, the documentation for them will need to be updated

@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Feb 11, 2025

  • Verify different tenant configs are working
  • Enable and verify emails are correct

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants