Project

General

Profile

Development #65839

Envoyer un courriel lors d'une création de compte via FranceConnect

Added by Valentin Deniaud 8 months ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
01 June 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

Revision 588fd616 (diff)
Added by Valentin Deniaud 3 months ago

auth_fc: send email on registration (#65839)

History

#7

Updated by Valentin Deniaud 3 months ago

  • Status changed from Nouveau to Information nécessaire
  • Assignee set to Mikaël Ates
#8

Updated by Mikaël Ates 3 months ago

  • Assignee changed from Mikaël Ates to Valentin Deniaud
#10

Updated by Valentin Deniaud 3 months ago

  • Status changed from Information nécessaire to En cours
#15

Updated by Valentin Deniaud 3 months ago

#16

Updated by Benjamin Dauvergne 3 months 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' %}).

#17

Updated by Valentin Deniaud 3 months 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.

#18

Updated by Benjamin Dauvergne 3 months 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.

#19

Updated by Valentin Deniaud 3 months 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)
#20

Updated by Transition automatique about 1 month ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF