Skip to content
This repository has been archived by the owner on Nov 1, 2024. It is now read-only.

Updating Google Tag Manager #12

Merged
merged 1 commit into from
Jan 25, 2024
Merged

Updating Google Tag Manager #12

merged 1 commit into from
Jan 25, 2024

Conversation

earth2marsh
Copy link
Contributor

@earth2marsh earth2marsh commented Jan 25, 2024 •
edited
Loading

removing the old AIP GTM container and establish an independent analytics property. This addresses #13 .

@earth2marsh
Updating Google Tag Manager
4fa71c0 
removing the old AIP GTM container
toumorokoshi
toumorokoshi approved these changes Jan 25, 2024
View reviewed changes
Copy link
Member

@toumorokoshi toumorokoshi left a comment

Choose a reason for hiding this comment

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

thanks for doing this! I think we should make the GA tag parameteriz-able at some point too.

@toumorokoshi toumorokoshi merged commit 414b8d6 into main Jan 25, 2024
4 checks passed
@toumorokoshi toumorokoshi mentioned this pull request Jan 25, 2024
Open
@rofrankel rofrankel deleted the earth2marsh-patch-1 branch January 25, 2024 22:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@toumorokoshi toumorokoshi toumorokoshi 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
@earth2marsh @toumorokoshi