Projet

Général

Profil

Bug #56755

Authentic2: préciser username s'il existe, dans le template de mail d'enregistrement

Ajouté par Nicolas Roche il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
07 septembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Lors d'un import d'utilisateur via CSV avec un colone @registration à send-email, les templates utilisés, api_user_create_registration_email_body.(txt|html), ne précise pas l'identifiant de l'utilisateur s'il existe.

Ces templates sont proches de new-account-notification_body.(txt|html), qui eux font bien mention du login s'il existe.
(tout comme le template natif qui précise également le username : src/authentic/src/authentic2/manager/templates/authentic2/manager/user_create_registration_email_body.txt)


Fichiers

Révisions associées

Révision 60f28dee (diff)
Ajouté par Nicolas Roche il y a plus de 2 ans

authentic2: provide username if available on registration mail (#56755)

Révision a6db34bf (diff)
Ajouté par Frédéric Péters il y a plus de 2 ans

Revert "authentic2: provide username if available on registration mail (#56755)"

This reverts commit 60f28dee2f81a281c4ac01b76dbfb3f73b64ac0f.

Historique

#2

Mis à jour par Nicolas Roche il y a plus de 2 ans

#3

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

  • Statut changé de Solution proposée à En cours

On pourrait faire pareil avec email parce que rien ne garantit se présence non plus je pense (et idem pour les champs suivants), je laisse ça pour un autre jour.

Par contre la façon dont sont placés les if ça va faire une ligne blanche en plus, surtout pour la version .txt, voir comment c'est fait dans le template d'origine (qui bizarre n'existe qu'en .txt..) https://git.entrouvert.org/authentic.git/tree/src/authentic2/templates/authentic2/api_user_create_registration_email_body.txt

#4

Mis à jour par Nicolas Roche il y a plus de 2 ans

Merci, je n'avais pas compris.
(je me suis aligné sur new-account-notification_body.txt )

#5

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

  • Statut changé de Solution proposée à Solution validée

Nicolas Roche a écrit :

Merci, je n'avais pas compris.
(je me suis aligné sur new-account-notification_body.txt )

Je ne dis pas que c'est bien fait partout, si tu y prends goût tu peux faire le tour des templates texte pour vérifier :)

On peut le faire dans le HTML aussi ça fait du html plus joli quand on regarde les sources, mais ça c'est juste pour le fun.

#6

Mis à jour par Nicolas Roche il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 60f28dee2f81a281c4ac01b76dbfb3f73b64ac0f
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Fri Sep 10 15:25:28 2021 +0200

    authentic2: provide username if available on registration mail (#56755)
#7

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF