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

✨Migrate Analytics to GA4 #289

Open
wants to merge 14 commits into
base: develop
Choose a base branch
from
Open

✨Migrate Analytics to GA4 #289

wants to merge 14 commits into from

Conversation

BennyBot
Copy link
Member

@BennyBot BennyBot commented May 26, 2023

Summary:

Migrates our existing UA analytics to the new GA4 Measurements Protocol. It is currently done I think, but there are problems with user location data and I'm not sure if it is a problem on the CLI's side or GA4's side.

Motivation:

UA will end within the next month

Test Plan:

  • Verify all commands have an event that appears on the realtime dashboard
  • Make sure user uuid generation is still intact
  • Verify that user location data is being streamed.

pros/ga/analytics.py Fixed Show fixed Hide fixed
@Ghurfa
Copy link

Ghurfa commented Oct 21, 2023

Verified that the following CLI commands show up on the google analytics real-time dashboard:

Tested Commands
  • pros lsusb
  • pros make
  • pros terminal
  • pros upgrade (Analytics succeeds, although it throws an error after)
  • pros upload
  • pros v5 capture
  • pros v5 cat-metadata
  • pros v5 ls-files
  • pros v5 read_variable
  • pros v5 rm-all
  • pros v5 rm-file
  • pros v5 rm-program
  • pros v5 run
  • pros v5 set_variable
  • pros v5 status
  • pros v5 stop
  • pros c apply
  • pros c create-template
  • pros c fetch
  • pros c info-project
  • pros c install
  • pros c new-project
  • pros c purge-template
  • pros c query-templates
  • pros c uninstall
  • pros c upgrade

It seems that it is not possible to stream location data using GA4 measurement protocol.

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

Successfully merging this pull request may close these issues.

3 participants