feat: add GDPR-compliant cookie consent banner #8022
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Created shared @roo-code/cookie-consent package with react-cookie-consent - Implemented CookieConsent component with proper styling (white bg, inverted in dark mode) - Integrated cookie consent into web-roo-code layout - Created basic Next.js app structure for web-docs - Integrated cookie consent into web-docs layout - Used shared component approach for consistency and maintainability
![roomote[bot]](https://avatars.githubusercontent.com/in/1546624?s=60&v=4){kind=small}
hannesrudolph
added
the
Issue/PR - Triage
Collaborator
|
@roomote-agent there's a lot of PR feedback for you to address here. |
Contributor
Author
|
Thanks for the feedback @brunobergher! I see the review comments and I'm on it. Here's what I'll be addressing:
I'll have these changes ready shortly! |
- Remove shared package approach, duplicate component in each app - Replace DOM manipulation with next-themes useTheme hook for dark mode - Replace all inline styles with Tailwind utility classes - Add proper ARIA attributes for accessibility - Remove unused js-cookie dependency - Align ESLint version to v9 across all apps Addresses all review comments from @brunobergher
Contributor
|
🚀 Preview deployed! Your changes have been deployed to Vercel: Preview URL: https://roo-code-website-hqgul24g8-roo-code.vercel.app This preview will be updated automatically when you push new commits to this PR. |
Contributor
Author
|
Hey @brunobergher! All requested changes have been implemented and pushed ✅ Changes Made:
CI Status:Most checks are passing. The Windows unit test failure appears to be a pre-existing issue unrelated to these changes (all other platform tests pass). The cookie consent implementation is now cleaner, more maintainable, and follows best practices for SSR/hydration. Ready for your review! |
brunobergher
changed the title
dosubot
bot
added
size:XL
mrubens
reviewed
Sep 16, 2025
apps/web-docs/.eslintrc.json
Outdated
Collaborator
There was a problem hiding this comment.
We should probably remove this apps/web-docs folder and everything in it
mrubens
reviewed
Sep 16, 2025
dosubot
bot
removed
the
size:XL
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds GDPR-compliant cookie banner to web-roo-code, ensuring cookie-driven analytics scripts are added only after consent.
Changes
Screenshots
Important
Adds GDPR-compliant cookie consent banner and updates analytics providers to respect user consent.
CookieConsentWrappercomponent inCookieConsentWrapper.tsxfor GDPR-compliant cookie consent.CookieConsentWrapperintolayout.tsxto display consent banner.GoogleAnalyticsProviderandPostHogProviderto load analytics only after consent.consent-manager.tsto handle consent events.CONSENT_COOKIE_NAMEandCOOKIE_CONSENT_EVENTSincookie-consent.ts.react-cookie-consentandtldtstopackage.json.This description was created by
for 9fc5f6c. You can customize this summary. It will automatically update as commits are pushed.