Project

General

Profile

Bug #56755

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

Added by Nicolas Roche about 1 month ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
07 Sep 2021
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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)


Files

Associated revisions

Revision 60f28dee (diff)
Added by Nicolas Roche about 1 month ago

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

Revision a6db34bf (diff)
Added by Frédéric Péters 27 days ago

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

This reverts commit 60f28dee2f81a281c4ac01b76dbfb3f73b64ac0f.

History

#2

Updated by Nicolas Roche about 1 month ago

#3

Updated by Benjamin Dauvergne about 1 month ago

  • Status changed from Solution proposée to 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

Updated by Nicolas Roche about 1 month ago

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

#5

Updated by Benjamin Dauvergne about 1 month ago

  • Status changed from Solution proposée to 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

Updated by Nicolas Roche about 1 month ago

  • Status changed from Solution validée to 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

Updated by Frédéric Péters about 1 month ago

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

Also available in: Atom PDF