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
fix(docs-infra): add cookie consent gtag event default state #54574
Conversation
2edf089
to
ac177a9
Compare
@bencodezen could you please also change the commit message to start with |
ac177a9
to
e23af8f
Compare
@AndrewKushnir Absolutely! Thanks for calling that out. I've updated the PR with the correct doc comment and message! |
e23af8f
to
f172f08
Compare
f172f08
to
c13a5b1
Compare
c13a5b1
to
9b92f99
Compare
@bencodezen LGTM, thanks for the updates. I've removed the "review" label, please add the "merge" label when the PR is ready for merge. Thank you. |
9b92f99
to
72b7549
Compare
72b7549
to
7ffc56b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for dev-infra (been waiting since this is blocked, but not ready for review— but want to get it off my list)
ffb3422
to
ec6d328
Compare
3154148
to
3226213
Compare
This PR sets a default state for cookie consent of 'denied'. The other part of the PR exists in the angular/dev-infra repo which will grant permission when the user accepts the cookie banner.
3226213
to
89fc523
Compare
Finally managed to get everything passing in the CI! 🥳 Ready for review and then merge! |
This PR was merged into the repository by commit 0b53fdb. |
This PR sets a default state for cookie consent of 'denied'. The other part of the PR exists in the angular/dev-infra repo which will grant permission when the user accepts the cookie banner. PR Close #54574
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
No Google Analytics consent code existed.
Issue Number: N/A
What is the new behavior?
Google Analytics code for cookie consent is now added per the docs.
The other half of this PR that grants cookie consent can be found at angular/dev-infra#1805
Does this PR introduce a breaking change?
Other information