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

Uses Google Analytics 4 tracking tag + removes Travis-CI config #69

Merged
merged 3 commits into from
Jul 10, 2024

Conversation

nkinkade
Copy link
Contributor

"Universal Analytics" (UA) is now retired, replaced by Google Analytics 4 (GA4). Prior to the retirement of UA, I linked our UA tracking code to a GA4 property, which allowed Analytics to continue to collect data using the old tracking code. However, this arrangement requires a couple extra calls to Analytics and reduces the feature set available to the property. This PR replaces the old UA tracking tag with the new GA4 tracking tag.

This PR also removes the Travis-CI config file, since all deployments are now handled through Github Actions on this repo.

nkinkade added 3 commits July 9, 2024 14:35
"Universal Analytics" is retired, replaced by Google Analytics 4 (GA4). Our UA
tags are linked to GA4 properties, but we should stop doing that and just use
the native GA4 analytics tag. This PR should do that.
The Travis-CI build is emitting this error:

"GLIBC_2.27 not found (required by node)"

Presumably the version of node we are using requires some version of glibc that
the default distro, Xenial, doesn't have. This is just an attempt to work
around this issue. It may cause other problems.
Github Actions on this repo does all the deployments now.
@nkinkade nkinkade requested a review from robertodauria July 10, 2024 17:36
@nkinkade nkinkade merged commit c486057 into main Jul 10, 2024
3 checks passed
@nkinkade nkinkade deleted the sandbox-kinkade branch July 10, 2024 20:03
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