-
Notifications
You must be signed in to change notification settings - Fork 3
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
Refresh page on consent change #1225
Conversation
🦋 Changeset detectedLatest commit: 8e0a65c The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
🚀 0.0.0-beta-20240124105308 published to npm as a beta release |
🚀 0.0.0-beta-20240124125623 published to npm as a beta release |
78e0d03
to
d63455b
Compare
🚀 0.0.0-beta-20240208122858 published to npm as a beta release |
5e05d5a
to
2759d28
Compare
🚀 0.0.0-beta-20240213111558 published to npm as a beta release |
46e4595
to
8f2390f
Compare
8f2390f
to
051a7f4
Compare
🚀 0.0.0-beta-20240227165933 published to npm as a beta release |
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.
Nice! Finally over the line!
}, true); | ||
}; | ||
|
||
const reloadPageOnConsentChange = once(() => |
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.
Not sure why we need to once() as I thought we would only call this in the boot script?
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.
Good spot - I think this is from the original PR back in frontend. I've removed it and seems to work fine without!
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.
🥳
ps @zekehuntergreen 😉
What does this change?
Adds a module to refresh the page when consent state changes, and an E2E test to wait for a page reload when consent state changes.
Also deletes the module to remove consented ads on consent withdrawal, as we no longer need this if we're refreshing the page immediately.
Why?
This means our data will correctly reflect the page consent status, and that users will immediately experience the appropriate ad experience on consent change.