Développement #65839
Envoyer un courriel lors d'une création de compte via FranceConnect
0%
Description
De #21914#note-5, Benjamin Dauvergne a écrit :
Dans le code actuel on envoie plus de mail d'enregistrement sur un enregistrement via FC, je vais ouvrir un ticket pour rétablir ça
Ce ticket est désormais ouvert.
Files
Associated revisions
History
Updated by Valentin Deniaud about 2 years ago
- Status changed from Nouveau to Information nécessaire
- Assignee set to Mikaël Ates
Updated by Valentin Deniaud about 2 years ago
- Status changed from Information nécessaire to En cours
Updated by Valentin Deniaud about 2 years ago
- File 0001-auth_fc-send-email-on-registration-65839.patch 0001-auth_fc-send-email-on-registration-65839.patch added
- Status changed from En cours to Solution proposée
- Patch proposed changed from No to Yes
Updated by Benjamin Dauvergne about 2 years ago
Pourquoi on n'envoie pas le mail de création classique1 tout bêtement ?
Je ne vois pas tellement de différence et le fait de pointer que c'est par FranceConnect ou pas ne me parait pas super utile.
1
{% load i18n %} <html> <body style="max-width: 90ex"> {% blocktrans %} <p> Hi {{ user }}!</p> <p>Your registration on {{ site }} was successful!</p> <p><a href="{{ login_url }}">Login!</a></p> {% endblocktrans %} <ul> {% if user.username %} <li>{% trans "Username:" %} {{ user.username }}</li> {% endif %} <li>{% trans "Email:" %} {{ user.email }}</li> {% if user.first_name %} <li>{% trans "First name:" %} {{ user.first_name }}</li> {% endif %} {% if user.last_name %} <li>{% trans "Last name:" %} {{ user.last_name }}</li> {% endif %} </ul> </body> </html>
et dans publik :
$ cat ./templates/authentic2/registration_success_body.html {% extends "emails/body_base.html" %} {% block content %} <p>Bonjour {{ user.get_full_name }},</p> <p> Votre compte a été créé. Pour vous connecter, utilisez les informations suivantes : </p> <ul> <li>{% if user.email %}Courriel : {{ user.email }}{% else %}Identifiant : {{ user.username }}{% endif %}</li> {% if user.has_usable_password %} <li>Mot de passe : celui que vous venez de choisir lors de l’inscription.</li> {% endif %} </ul> {% include "emails/button-link.html" with url=login_url label="Me connecter maintenant" %} {% endblock %}
On pourrait passer juste une variable method
pour spécialiser le texte dans certains cas (puisqu'ici ça précise qu'il faut utiliser son email pour se connecter il faudrait le conditionner avec {% if method == 'email' %}
).
Updated by Valentin Deniaud about 2 years ago
Benjamin Dauvergne a écrit :
Pourquoi on n'envoie pas le mail de création classique1 tout bêtement ?
Pour les raisons que tu indiques : réutiliser le mail de création ça implique de rajouter des conditions ({% if method == 'email' %}
) et de se poser la question d'où il est surchargé et d'aller introduire la condition à ces endroits (là heureusement il n'y a que dans templates/ de publik-base-theme et pas d'inté graphiques).
Face à ça ça m'a paru nettement plus bête d'en écrire un nouveau, pas de surprises et comme le mail est très court ça ne fait pas beaucoup de lignes en plus à maintenir (et les lignes sont simples contrairement à des lignes avec des conditions).
Je ne vois pas tellement de différence et le fait de pointer que c'est par FranceConnect ou pas ne me parait pas super utile.
Soit mais dans le ticket que je pointe en description il y a écrit
ça pourrait être bien de varier le message entre quelqu'un qui s'est créé un compte de manière classique et quelqu'un qui s'est créé un compte via FranceConnect
et donc je partait sur cette base, avec laquelle Mik semblait d'accord.
Updated by Benjamin Dauvergne about 2 years ago
- Status changed from Solution proposée to Solution validée
Valentin Deniaud a écrit :
Face à ça ça m'a paru nettement plus bête d'en écrire un nouveau, pas de surprises et comme le mail est très court ça ne fait pas beaucoup de lignes en plus à maintenir (et les lignes sont simples contrairement à des lignes avec des conditions).
Bon vu qu'aucun thème ne surcharge le template de base dans publik-base-theme (ce qui me parait étonnant mais bon) ça ne crée pas vraiment de boulot supplémentaire dans un cas comme dans l'autre, alors ok.
Updated by Valentin Deniaud about 2 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit 588fd6162873e6db6d199c6fd1d59ecdcffb026e Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Thu Nov 3 16:38:44 2022 +0100 auth_fc: send email on registration (#65839)
Updated by Transition automatique almost 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
auth_fc: send email on registration (#65839)