-
Notifications
You must be signed in to change notification settings - Fork 5
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
Cookie policy lists update, third-parties config tweaked #10896
Conversation
Size Change: +77 B (+0.01%) Total Size: 971 kB
ℹ️ View Unchanged
|
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.
Looks good. The only strangeness here is that we have the cookie name for necessary cookies only but not for the other categories. Not a blocker for deployment and perhaps the name field could be optional where it's a third party cookie?
Agreed, I think we need to discuss this with @LaurenFBaily first thing on Tuesday. I've added it to the release plan and tagged her so we make sure it happens. |
Who is this for?
Cookies work #10895 and #10717
What is it doing for them?
Table
so text is smaller/kind of more readable.