Skip to content

feat: add GDPR cookie consent banner for Google Analytics#7310

Open
kaminimangal wants to merge 2 commits intolouislam:masterfrom
kaminimangal:feature/gdpr-cookie-consent
Open

feat: add GDPR cookie consent banner for Google Analytics#7310
kaminimangal wants to merge 2 commits intolouislam:masterfrom
kaminimangal:feature/gdpr-cookie-consent

Conversation

@kaminimangal
Copy link
Copy Markdown

Summary

In this pull request, the following changes are made:
Added a cookie consent toggle in status page settings.
When enabled with google analytics, it shows a banner asking whether it has to accept or decline cookies.
Used localStorage to remember the user's choice

#4337 (comment)

Please follow this checklist to avoid unnecessary back and forth (click to expand)
  • ⚠️ If there are Breaking change (a fix or feature that alters existing functionality in a way that could cause issues) I have called them out
  • 🧠 I have disclosed any use of LLMs/AI in this contribution and reviewed all generated content.
    I understand that I am responsible for and able to explain every line of code I submit.
    I used Claude AI to understand the codebase and debug but I wrote and tested the most of code by myself.
  • 🔍 Any UI changes adhere to visual style of this project.
  • 🛠️ I have self-reviewed and self-tested my code to ensure it works as expected.
  • 📝 I have commented my code, especially in hard-to-understand areas (e.g., using JSDoc for methods).
  • 🤖 I added or updated automated tests where appropriate.
  • 📄 Documentation updates are included (if applicable).
  • 🧰 Dependency updates are listed and explained.
  • ⚠️ CI passes and is green.

Screenshots for Visual Changes

image image image image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant