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

Feat/dt 1927 Adding GA4 server side #199

Merged
merged 1 commit into from
Jun 21, 2024
Merged

Conversation

ian-leggett
Copy link
Contributor

@ian-leggett ian-leggett commented Apr 2, 2024

This changes adds the GA4 server side.

I am keeping the old implementation for two reasons:

  1. We can run them side by side to make sure the data is the same across both UA and GA4.
  2. To make sure that the new implementation works. I can't test this from my localhost.

@ian-leggett ian-leggett marked this pull request as draft April 2, 2024 10:48
@ian-leggett ian-leggett force-pushed the feat/dt-1927-cookie-banner branch 3 times, most recently from ffe4b3d to 642831e Compare June 20, 2024 15:55
@ian-leggett ian-leggett changed the title WIP Feat/dt 1927 cookie banner Feat/dt 1927 Adding GA4 server side Jun 20, 2024
@ian-leggett ian-leggett marked this pull request as ready for review June 20, 2024 16:02
@ian-leggett ian-leggett requested a review from a team as a code owner June 20, 2024 16:02
app.py Outdated Show resolved Hide resolved
@michalc michalc force-pushed the feat/dt-1927-cookie-banner branch 9 times, most recently from 1b1a2f9 to dea0064 Compare June 21, 2024 10:08
This adds GA4 server-side analytics using the GA4 measurement protocol, similar
to the UA measurement protocol already in place.

Both will be run at the same time for now as GA4 is tested - and there may well
be more changes to the GA4 server-side analytics in the next week or so.

Note that server-side is fairly important because the primary function of the
public-data-api is the API (which includes data download), which do not have
web pages in front of them to run client-side analytics.

Co-authored-by: Ian Leggett <[email protected]>
Co-authored-by: Michal Charemza <[email protected]>
@michalc michalc force-pushed the feat/dt-1927-cookie-banner branch from dea0064 to c850421 Compare June 21, 2024 10:31
@michalc michalc merged commit 91d32e9 into main Jun 21, 2024
1 check passed
@michalc michalc deleted the feat/dt-1927-cookie-banner branch June 21, 2024 10:50
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.

2 participants