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

Mentions légales et opt-out pour haie #468

Merged
merged 5 commits into from
Nov 18, 2024
Merged

Mentions légales et opt-out pour haie #468

merged 5 commits into from
Nov 18, 2024

Conversation

@thibault thibault requested a review from pyDez November 7, 2024 10:03
Copy link
Collaborator

@pyDez pyDez left a comment

Choose a reason for hiding this comment

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

A mon tour de pinailler un peu :)


<h2>Directeurs ou directrices de la publication</h2>

<p>M. Philippe MAZENC, directeur général de l'aménagement, du logement et de la nature (DGALN).</p>
Copy link
Collaborator

Choose a reason for hiding this comment

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

On ne peut pas éviter d'avoir des noms de personnes susceptibles d'évoluer dans le temps ? C'est casse pied à maintenir...

Copy link
Collaborator Author

@thibault thibault Nov 18, 2024

Choose a reason for hiding this comment

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

Je crois qu'il est obligatoire d'avoir un nom de personne physique. Dans tous les cas, le texte est fourni tel quel par les juristes de chez beta, donc j'éviterais de trop le changer si je peux.


{% if visitor_id %}
<p>
Nous réalisons également un suivi statistique de certaines actions de navigation (par exemple les étapes d’utilisation du simulateur ou l’utilisation de l’outil de saisie du linéaire de haie) qui sont stockées localement et de manière anonyme, afin de mieux comprendre l'utilisation du service.
Copy link
Collaborator

Choose a reason for hiding this comment

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

"qui sont stockées localement" Le cookie est certe stocké localement, mais les actions de navigation sont stockées chez nous.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Effectivement, je ne sais plus d'où vient cette phrase. Je modifie.

{% endif %}

<h2>Licence</h2>

Copy link
Collaborator

Choose a reason for hiding this comment

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

On ne redit pas ici : "Sauf mention contraire, tous les textes de ce site sont sous licence etalab-2.0" ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ce texte est présent dans le footer il me semble.

path(_("legal-mentions/"), LegalMentionsView.as_view(), name="legal_mentions"),
path(
_("legal-mentions/"),
PrivacyView.as_view(template_name="haie/pages/legal_mentions.html"),
Copy link
Collaborator

Choose a reason for hiding this comment

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

C'est pratique de réutiliser la PrivacyView ici, mais je trouve que le nom nous induit en erreur.

Peut-etre faire une VisitorTemplateView que l'on utiliserait ici et dont la PrivacyView heriterait ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok, j'ai complètement viré la vue qui ne servait pas à grand chose. Je passe par un context processor pour passer la variable au template.

@thibault thibault requested a review from pyDez November 18, 2024 10:48
envergo/analytics/views.py Dismissed Show dismissed Hide dismissed
Copy link
Collaborator

@pyDez pyDez left a comment

Choose a reason for hiding this comment

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

👌

@thibault thibault merged commit a0eb413 into main Nov 18, 2024
7 of 8 checks passed
@thibault thibault deleted the legal_mentions branch November 18, 2024 14:05
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.

2 participants