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

Run cookie policy before GTM #373

Merged
merged 2 commits into from
Oct 24, 2024
Merged

Run cookie policy before GTM #373

merged 2 commits into from
Oct 24, 2024

Conversation

britneywwc
Copy link
Contributor

@britneywwc britneywwc commented Oct 11, 2024

Done

  • Invoke cookie policy before google tag manager

QA

[List of steps to QA the new features or prove the bug has been resolved]

Issue / Card

[List of links to Github issues/bugs and cards if needed - e.g. Fixes #1]

Screenshots

[if relevant, include a screenshot]

Fixes https://warthogs.atlassian.net/browse/WD-15825

@webteam-app
Copy link
Collaborator

@laszlokajtar
Copy link

@laszlokajtar
Copy link

consent info is not present in the dataLayer
image

Copy link

@laszlokajtar laszlokajtar left a comment

Choose a reason for hiding this comment

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

lgtm!
image

@britneywwc britneywwc merged commit 116e955 into main Oct 24, 2024
6 checks passed
@britneywwc britneywwc deleted the cookie-gtm branch October 24, 2024 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants