chore: remove legacy CookieMessage component#1024
Conversation
The CookieMessage component was used for the classic cookie consent banner on the Ably website. The website now exclusively uses OneTrust for cookie consent management, making this component unused. Companion change: ably/website#chore/remove-classic-cookie-consent
WalkthroughThe CookieMessage component and all its associated files have been removed from the codebase, including the React component, Storybook story, CSS styling, and stylesheet imports. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
📜 Recent review detailsConfiguration used: Repository UI Review profile: CHILL Plan: Pro Disabled knowledge base sources:
📒 Files selected for processing (4)
💤 Files with no reviewable changes (4)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
The CookieMessage component was used for the classic cookie consent banner on the Ably website. The website now exclusively uses OneTrust for cookie consent management, making this component unused.
Companion change: https://github.com/ably/website/pull/7679
This was discovered during local development when I saw the old cookie banner pop up, which was confusing. Turns out we added OneTrust years ago, but never removed the old code.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.