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 personal data link in footer #574

Merged
merged 4 commits into from
Oct 11, 2024

Conversation

juggler31
Copy link

@juggler31 juggler31 commented Sep 26, 2024

#527

Cette PR a pour but de rajouter un lien dans le footer, faisant apparaître une modale .
Le texte contenu dans cette modale est surchargeable via un fichier personal-data.html

Il a pour but de renseigner les utilisateurs sur les données personnelles comme indiqué sur le site de la CNIL : https://www.cnil.fr/fr/exemples-de-formulaire-de-collecte-de-donnees-caractere-personnel

La modale par défaut:

image

@juggler31 juggler31 force-pushed the rgpd_link_personal_data branch 2 times, most recently from 5c7ced0 to da2ab75 Compare September 26, 2024 14:47
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.

C'est globalement bon pour moi 👍

Je t'ai fait quelques remarques mineures. Pour les deux remarques qui concernent du code supprimé je pense qu'il peut être pertinent de le faire dans deux commits séparés avec une brêve explication dans les messages de commit. À toi de voir. :)

atlas/static/css/atlas.css Outdated Show resolved Hide resolved
atlas/babel.cfg Show resolved Hide resolved
atlas/configuration/config.py.example Outdated Show resolved Hide resolved
Le script ne fonctionnait plus depuis le commit : PnX-SI@c897d55
Les fichiers `.po`, `.mo`, et `.pot` n'était pas à jours, ce commit les met juste à jours.
@juggler31 juggler31 force-pushed the rgpd_link_personal_data branch 2 times, most recently from 30c094c to 864296a Compare September 27, 2024 08:41
@juggler31 juggler31 force-pushed the rgpd_link_personal_data branch from 864296a to 8c57c84 Compare September 27, 2024 09:56
@juggler31 juggler31 force-pushed the rgpd_link_personal_data branch from 8c57c84 to 61d733e Compare September 30, 2024 09:20
@juggler31 juggler31 requested a review from submarcos October 1, 2024 14:46
@submarcos
Copy link
Member

il manquerait pas un changelog pour expliquer ? ou une doc

@juggler31 juggler31 force-pushed the rgpd_link_personal_data branch from 61d733e to 7f0eae0 Compare October 4, 2024 09:59
@amandine-sahl amandine-sahl merged commit 002c6ee into PnX-SI:develop Oct 11, 2024
1 check passed
amandine-sahl pushed a commit that referenced this pull request Oct 15, 2024
* fix: remove script in footer.html template
Le script ne fonctionnait plus depuis le commit : c897d55

* fix: remove babel extensions because they are integrated in package now

* fix: launch translation cmd

Les fichiers `.po`, `.mo`, et `.pot` n'était pas à jours, ce commit les met juste à jours.

* feat: add personal data link in footer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants