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.

Sign up for GitHub

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

Jump to bottom

Migrate analytics to GA4 #790

Merged
merged 1 commit into from
Apr 7, 2023
Merged

Migrate analytics to GA4 #790

merged 1 commit into from
Apr 7, 2023

Conversation

marlo-longley
Copy link
Contributor

@marlo-longley marlo-longley commented Apr 4, 2023 •
edited
Loading

Part of #759

@marlo-longley
Copy link
Contributor Author

marlo-longley commented Apr 4, 2023 •
edited
Loading

I guess this fails codeclimate. I can extract some functions to reduce repetition, but not sure there is a ton of benefit in this case.

@marlo-longley marlo-longley marked this pull request as ready for review April 4, 2023 16:23
@thatbudakguy thatbudakguy merged commit a963597 into main Apr 7, 2023
@thatbudakguy thatbudakguy deleted the ga4 branch April 7, 2023 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thatbudakguy thatbudakguy thatbudakguy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@marlo-longley @thatbudakguy