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.

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

Add Cookie consent modal #467

Open
peterjurco opened this issue Nov 7, 2024 · 3 comments · Fixed by #508
Open

Add Cookie consent modal #467

peterjurco opened this issue Nov 7, 2024 · 3 comments · Fixed by #508

Comments

@peterjurco
Copy link
Collaborator

peterjurco commented Nov 7, 2024

As preparation for adding Google Analytics and Clarity to the DAO dashboard we want to add user consent for analytical cookies (along the error reporting ones) to the DAO Dashboard. Clicking on "Manage Settings" user should see the Privacy settings modal similar to the one on the Market. This should apply only to the new rebranded version.

Design: https://www.figma.com/design/NG6QClnVrvPRZ5gJ3s0zen/%E2%9F%81--Api3-DAO---Branding-Update?node-id=9704-161128&node-type=frame&m=dev

Image

Image

@Anboias Anboias self-assigned this Nov 22, 2024
@Anboias Anboias linked a pull request Nov 29, 2024 that will close this issue
@PrenaamD
Copy link
Collaborator

PrenaamD commented Dec 3, 2024

@Anboias
According to the design the cookie banner should read
In order to provide the best services for you, we collect anonymized error data through [Sentry](https://sentry.io/) and use analytics cookies to improve our products.
The implementation reads In order to provide the best services for you, we collect anonymized error data through [Sentry](https://sentry.io/). We do not gather IP address or user agent information.

@Anboias
Copy link
Collaborator

Anboias commented Dec 3, 2024

@PrenaamD, yep. Copied it 1on1 from the Market, felt like that's the best way to go. I'll ask the team about it.

@Anboias
Copy link
Collaborator

Anboias commented Dec 3, 2024

@PrenaamD
I was actually looking in the wrong place, my bad. You were right. Corrected.

@PrenaamD PrenaamD added the TESTED label Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants