Add page to manage cookie preferences #1213
Conversation
6fbc9d5
to
7ebdc53
Compare
Looking at this now. |
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.
Straightforward, working implementation. I suggested DRYing it up a bit.
<div> | ||
The Open Apparel Registry uses cookies to collect and analyze | ||
site performance and usage. By clicking the Accept button, you | ||
agree to allow us to place cookies and share information with | ||
Google Analytics. For more information, please visit our{' '} | ||
<a | ||
href="https://info.openapparel.org/tos/" | ||
target="_blank" | ||
rel="noopener noreferrer" | ||
> | ||
Terms and Conditions of Use and Privacy Policy. | ||
</a> | ||
</div> |
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.
Consider extracting this tex into a component that can be used both here and in GDPRNotification.jsx
. If we need to update this language in the future it will eliminate the possibility of accidentally updating it in one place and not another.
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 call! Thank you for the review!
Gives the user the option to access a cookies preferences page where they can choose to change the opt in or out preference that they set when first visiting OAR.
7ebdc53
to
51a8f07
Compare
Overview
Gives the user the option to access a cookies preferences page where
they can choose to change the opt in or out preference that they set
when first visiting OAR.
Connects #1191
Demo
Testing Instructions
./scripts/server
Checklist
fixup!
commits have been squashed