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

Invoke cookie policy before gtm #303

Merged
merged 1 commit into from
Oct 15, 2024
Merged

Invoke cookie policy before gtm #303

merged 1 commit into from
Oct 15, 2024

Conversation

britneywwc
Copy link
Contributor

Done

  • Call cookie policy script before invoking gtm

QA

  • Check out this feature branch
  • Run the site using the command ./run serve
  • View the site locally in your web browser at: http://0.0.0.0:8037
  • Run through the following QA steps
  • [List additional steps to QA the new features or prove the bug has been resolved]

Issue / Card

Fixes #

Screenshots

[if relevant, include a screenshot]

@webteam-app
Copy link

Demo

Jenkins

demos.haus

laszlokajtar
laszlokajtar approved these changes Oct 14, 2024
View reviewed changes
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!
Uploading image.png…

@britneywwc britneywwc merged commit e9299cd into main Oct 15, 2024
6 checks passed
@britneywwc britneywwc deleted the gtm-cookie branch October 15, 2024 03:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@laszlokajtar laszlokajtar laszlokajtar approved these changes

@immortalcodes immortalcodes immortalcodes approved these changes

Assignees
No one assigned
Labels
Review: QA +1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@britneywwc @webteam-app @immortalcodes @laszlokajtar