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

Replace Osano cookie notification code with original code #69

Closed
delphidabbler opened this issue Sep 25, 2022 · 3 comments
Closed

Replace Osano cookie notification code with original code #69

delphidabbler opened this issue Sep 25, 2022 · 3 comments
Assignees
Labels
completed Work on issue completed. Awaiting closure.

Comments

@delphidabbler
Copy link
Owner

The current cookie management code is from the Osano free plan. This is not true open source.

It would be better to either get a true open source replacement or write some original code.

@delphidabbler delphidabbler self-assigned this Sep 25, 2022
@delphidabbler delphidabbler added update needed A file or feature needs bringing up to date accepted Accepted this issue as a valid issue that will be actioned labels Sep 25, 2022
@delphidabbler
Copy link
Owner Author

Ensure revised code meets EU regulations:

  • The cookies used are preferences cookies. Strictly speaking the user can opt out of these. So a Yes/No choice is required. If no then the cookie compliance popup & any flash messages will always appear.

  • Cookies can't be stored for > 1 year, so permanently hiding flash messages is not an option.

  • Users decisions to opt out must be recorded - let's argue that this is done by storing the cookie.

  • Information about each cookie used must be provided, so need to update cookies.html. For accuracy it may be a good idea to store each cookie's name in a Jekyll variable so same one gets used as actual name & in Docs.

  • Users must be able to withdraw consent. Possibly add a link to click on page footer when user is opted in.

@delphidabbler
Copy link
Owner Author

delphidabbler commented Sep 28, 2022

Wait for delphidabbler/delphidabbler.github.io implementation to be completed and adapt that.

See: delphidabbler/delphidabbler.github.io#129

@delphidabbler
Copy link
Owner Author

Implemented at commit 13b53a1

@delphidabbler delphidabbler added completed Work on issue completed. Awaiting closure. and removed accepted Accepted this issue as a valid issue that will be actioned update needed A file or feature needs bringing up to date labels Nov 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
completed Work on issue completed. Awaiting closure.
Projects
None yet
Development

No branches or pull requests

1 participant