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

feat: Add orejime to manage cookies #583

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

juggler31
Copy link

Ajout d'un gestionnaire de cookies. Orejime a été choisi. Il permet d'avoir une popup afin d'accepter ou de refuser les cookies sur l'application.

image

Le lien de la documentation du paquet orejime servant a accepter ou refuser les cookies sur l'application : https://github.com/empreinte-digitale/orejime

@juggler31 juggler31 force-pushed the add_orejime_manager branch 3 times, most recently from 2c515e3 to 7ac2c17 Compare October 17, 2024 13:21
@camillemonchicourt
Copy link
Member

Issue liée : #527

Copy link

@marcantoinedupre marcantoinedupre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, je t'ai fait quelques retours dans le code.

Et une remarque plus globale : il faudrait ajouter les nouveaux paramètres de configuration (+ une description succincte de chaque paramètre) dans config.py.example.

@@ -0,0 +1,139 @@
La RGPD

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le RGPD

Mais je pense qu'un titre et nom de fichier « Gestion des cookies » ou « Acceptation des cookies » serait peut-être plus pertinent pour cette section. Qu'en penses-tu ?

logo: false,
debug: configuration.modeDebug,
translations: configuration.OREJIME_TRANSLATIONS,
apps: configuration.OREJIME_APPS,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il faudrait vérifier ce qui se passe quand les paramètres OREJIME ne sont pas définis dans config.py. Ce qui sera le cas pour les instances déjà existantes.

@juggler31
Copy link
Author

Il manque un bouton afin de pouvoir changer le choix des cookies déjà renseignés.

@juggler31 juggler31 marked this pull request as draft November 15, 2024 09:11
@juggler31 juggler31 mentioned this pull request Nov 27, 2024
@juggler31 juggler31 marked this pull request as ready for review November 27, 2024 10:26
Ajout d'un gestionnaire de cookies. Orejime a été choisi.
Il permet d'avoir une popup afin d'accepter ou de refuser les cookies sur l'application.
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.

3 participants