Development #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.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Nouveau à Information nécessaire
- Assigné à mis à Mikaël Ates (de retour le 29 avril)
Mis à jour par Mikaël Ates (de retour le 29 avril) il y a plus d'un an
- Assigné à changé de Mikaël Ates (de retour le 29 avril) à Valentin Deniaud
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Information nécessaire à En cours
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-auth_fc-send-email-on-registration-65839.patch 0001-auth_fc-send-email-on-registration-65839.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus d'un an
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' %}).
Mis à jour par Valentin Deniaud il y a plus d'un an
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.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution proposée à 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.
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Solution validée à 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)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
auth_fc: send email on registration (#65839)