-
Notifications
You must be signed in to change notification settings - Fork 3
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 banner #9
Comments
I vote nuke cookies. Those banners are the worst, and it's not like we use cookies for anything, right? If we do use cookies for stuff, and instead switch to local storage, do we have to display a banner? |
I also vote nuke cookies to the extent it is possible. What cookies do we use and when? Here is quite good information about the cookie law: https://webbriktlinjer.se/lagkrav/kaklagen-i-praktiken/ A relevant part is that we might not need to nuke all cookies. And that all ways to track users are covered by the law, so we cannot simply change to LocalStorage and say that we're compliant without getting consent from the users. |
I think the cookies mainly are used to select language, which would be fine. Although I'd also vote to try to nuke 'em if it's not to complicated. I mean we could use the path or sub-domains to select locale. Or are cookies preferable over path to set the locale?
Are there more things to consider? |
The site uses cookies apparently, so we should either add an annoying cookie banner or try to nuke the cookies
The text was updated successfully, but these errors were encountered: