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

Setup Google Tag Manager #8046

Merged
merged 1 commit into from
Aug 9, 2023

Conversation

dnkrj
Copy link
Contributor

@dnkrj dnkrj commented Aug 2, 2023

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

What

Add code for Google Tag Manager

https://trello.com/c/EDrVRCsv/1515-set-up-ga4-property-and-gtm-container

Why

Needed as part of the process of moving to GA4

app/views/layouts/admin.html.erb Outdated Show resolved Hide resolved
app/views/layouts/admin.html.erb Outdated Show resolved Hide resolved
In advance of moving to GA4
@dnkrj dnkrj force-pushed the 1515-set-up-ga4-property-and-gtm-container branch from c9f8e13 to 896e748 Compare August 8, 2023 08:28
dnkrj added a commit to alphagov/govuk-helm-charts that referenced this pull request Aug 8, 2023
In advance of GA4 we're adding Tag Manager to whitehall:
alphagov/whitehall#8046

In keeping with Content Publisher the application code looks for an environment variable for the id - which this PR provides.

https://trello.com/c/EDrVRCsv/1515-set-up-ga4-property-and-gtm-container
ollietreend pushed a commit to alphagov/govuk-helm-charts that referenced this pull request Aug 9, 2023
In advance of GA4 we're adding Tag Manager to whitehall:
alphagov/whitehall#8046

In keeping with Content Publisher the application code looks for an environment variable for the id - which this PR provides.

https://trello.com/c/EDrVRCsv/1515-set-up-ga4-property-and-gtm-container
Copy link
Contributor

@ollietreend ollietreend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 👍🏻

@ollietreend ollietreend merged commit 5c87f6e into main Aug 9, 2023
@ollietreend ollietreend deleted the 1515-set-up-ga4-property-and-gtm-container branch August 9, 2023 13:24
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