Projet

Général

Profil

Development #19566

Ajouter les mails en cas de réussite de la création de compte

Ajouté par Benjamin Dauvergne il y a plus de 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
19 octobre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Les mails de base dans a2:

+++ b/src/authentic2/templates/authentic2/registration_success_body.html
@@ -0,0 +1,23 @@
+{% load i18n %}
+<html>
+  <body style="width: 90ex">
+    {% blocktrans with full_name=user.get_full_name %}
+        <p> Hi {{ full_name }} !</p>
+        <p>Your registration on {{ site }} was successfull !</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>
diff --git a/src/authentic2/templates/authentic2/registration_success_body.txt b/src/authentic2/templates/authentic2/registration_success_body.txt
new file mode 100644
index 0000000..9da1a14
--- /dev/null
+++ b/src/authentic2/templates/authentic2/registration_success_body.txt
@@ -0,0 +1,14 @@
+{% load i18n %}{% autoescape off %}{% blocktrans with full_name=user.get_full_name %} Hi {{ full_name }} !
+
+Your registration on {{ site }} was successfull !
+
+You can login on:
+
+       {{ login_url }}
+{% endblocktrans %}{% if user.username %}
+       {% trans "Username:" %} {{ user.username %}{% endif %}
+       {% trans "Email:" %} {{ user.email }}{% if user.first_name %}
+       {% trans "First name:" %} {{ user.first_name }}{% endif %}{% if user.last_name %}
+       {% trans "Last name:" %} {{ user.last_name }}{% endif %}
+
+{% endautoescape %}
diff --git a/src/authentic2/templates/authentic2/registration_success_subject.txt b/src/authentic2/templates/authentic2/registration_success_subject.txt
new file mode 100644
index 0000000..cfc922c
--- /dev/null
+++ b/src/authentic2/templates/authentic2/registration_success_subject.txt
@@ -0,0 +1 @@
+{% load i18n %}{% autoescape off %}{% trans "You successfully registered on" %} {{ site }}{% endautoescape %}

Fichiers


Demandes liées

Lié à Authentic 2 - Development #19416: Envoyer un mail après un enregistrement réussiFermé13 octobre 2017

Actions

Révisions associées

Révision ad488faa (diff)
Ajouté par Benjamin Dauvergne il y a plus de 6 ans

templates: add registration_success email templates (#19566)

Révision d3ac30f7 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 6 ans

templates: fix unclosed block in registration_success_body (#19566)

Historique

#1

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

#3

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

Pour vous connecter veuillez cliquer

Dans les autres modèles similaires on a une virgule après "Pour vous connecter".

À votre demande

Ça me semble bizarre pour quelque chose que l'usager a fait, plutôt que demandé.

#4

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

Correction suite à commentaire de Mike et relecture par moi même:
  • erreur de template dans body.html
  • ajout virgule manquante dans body.txt
#5

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

Frédéric Péters a écrit :

Pour vous connecter veuillez cliquer

Dans les autres modèles similaires on a une virgule après "Pour vous connecter".

Yep je prends.

À votre demande

Ça me semble bizarre pour quelque chose que l'usager a fait, plutôt que demandé.

Ok, mais je mets quoi ? (je déteste faire des templates :) )

#7

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

Ça me semble bizarre pour quelque chose que l'usager a fait, plutôt que demandé.

Ok, mais je mets quoi ? (je déteste faire des templates :) )

Du coup j'ai fait un tour dans ma boite et ce que je proposerais :

  Bonjour {{prenom}} {{nom}},

  Votre compte a été créé : bienvenue !  Pour vous connecter, utilisez les informations
  suivantes :

    * {% if username %}Identifiant : {{username}}{% else %}Courriel : {{email}}{% endif %}
    * Mot de passe : celui que vous venez de choisir lors de l'inscription.

  [ bouton "Me connecter maintenant" ]
#8

Mis à jour par Mikaël Ates il y a plus de 6 ans

Ok pour moi.

#9

Mis à jour par Thomas Noël il y a plus de 6 ans

"À votre demande, un compte vient de vous être créé :" ça le fait pas, on envoie juste avant un "Vous venez de créer votre compte, nous vous en remercions." dans le mail d'activation.

Il faut parler de compte activé, quelque chose comme ça. Peut-être :

Peut-être :

La création de votre compte est effectuée. (ou "terminée", "validée" ?). Pour rappel, voici ses coordonnées : ("données", "valeurs" ?)

...

et dans le sujet, reprendre "création de compte terminée/validée/whatever"

(Je veux bien l'avis d'autres, je suis pas inspiré)

#10

Mis à jour par Thomas Noël il y a plus de 6 ans

Update : la proposition de Fred me va, sauf le "bienvenue !" (j'aime pas qu'un robot me dise bonjour)

#11

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

Thomas Noël a écrit :

"À votre demande, un compte vient de vous être créé :" ça le fait pas, on envoie juste avant un "Vous venez de créer votre compte, nous vous en remercions." dans le mail d'activation.

Il faut parler de compte activé, quelque chose comme ça. Peut-être :

Peut-être :

[...]

et dans le sujet, reprendre "création de compte terminée/validée/whatever"

(Je veux bien l'avis d'autres, je suis pas inspiré)

Tout à fait d'accord avec toi mais c'est le mail d'activation qui devrait changer aussi,

<p>
Vous venez de créer votre compte, nous vous en remercions.
</p>

Ça c'est faut en fait, on a rien créé, il faudrait plutôt:

<p>
Vous demandez la création d'un compte, nous vous en remercions.
</p>

Et je ne sais pas vraiment pourquoi on les remercie.

#12

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

J'ai repris la proposition de Fred, petite modification j'appelle le courriel un identifiant parce que c'est ce qu'il devient ici. J'ai modifié le sujet du mail pour "création de votre compte terminée".

#13

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

    * Identifiant : {% if user.username %}{{ user.username }}{% else %}{{ user.email }}{% endif %}

Mais je suggérais :

    * {% if username %}Identifiant : {{username}}{% else %}Courriel : {{email}}{% endif %}

Qui préfixe différemment l'attribut "email", et produit un contenu similaire à new-account-notification_body.

Par rapport à ce template, et /api_user_create_registration_email_body, il y a également l'indentation des puces qui n'est pas conforme.

#14

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

J'ai repris la proposition de Fred, petite modification j'appelle le courriel un identifiant parce que c'est ce qu'il devient ici. J'ai modifié le sujet du mail pour "création de votre compte terminée".

Et comme je réagis là-dessus, je prône la cohérence avec nos autres templates, et le libellé qu'on utilise pour ce champ "identifiant" sur le formulaire de login, où il est "Courriel" (ex: https://connexion-strasbourg.test.entrouvert.org/login/)

#15

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

Diff incrémental, j'avais effectivement oublié une balise, j'ai remis la distinction Identifiant/Courriel

diff --git a/templates/authentic2/registration_success_body.html b/templates/authentic2/registration_success_body.html
index ffecdf8..dfeb201 100644
--- a/templates/authentic2/registration_success_body.html
+++ b/templates/authentic2/registration_success_body.html
@@ -8,7 +8,7 @@ informations suivantes :
 </p>

 <ul>
-  <li>Identifiant : {% if user.username %}{{ user.username }}{% else %}{{ user.email }}</li>
+  <li>{% if user.username %}Identifiant : {{ user.username }}{% else %}Courriel : {{ user.email }}{% endif %}</li>
   <li>Mot de passe : celui que vous venez de choisir lors de l'inscription.</li>
 </ul>

diff --git a/templates/authentic2/registration_success_body.txt b/templates/authentic2/registration_success_body.txt
index 30702a6..455fad8 100644
--- a/templates/authentic2/registration_success_body.txt
+++ b/templates/authentic2/registration_success_body.txt
@@ -3,7 +3,7 @@
 Votre compte a été créé : bienvenue !  Pour vous connecter, utilisez les
 informations suivantes :

-       * Identifiant : {% if user.username %}{{ user.username }}{% else %}{{ user.email }}{% endif %}
+       * {% if user.username %}Identifiant : {{ user.username }}{% else %}Courriel : {{ user.email }}{% endif %}
        * Mot de passe : celui que vous venez de choisir lors de l'inscription.

 Pour vous connecter, veuillez cliquer sur le lien suivant :
#16

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

Par rapport à ce template, et /api_user_create_registration_email_body, il y a également l'indentation des puces qui n'est pas conforme.

#17

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

J'ai revu l'indentation (diff incrémental):

diff --git a/templates/authentic2/registration_success_body.txt b/templates/authentic2/registration_success_body.txt
index 455fad8..eff9ceb 100644
--- a/templates/authentic2/registration_success_body.txt
+++ b/templates/authentic2/registration_success_body.txt
@@ -3,10 +3,10 @@
 Votre compte a été créé : bienvenue !  Pour vous connecter, utilisez les
 informations suivantes :

-       * {% if user.username %}Identifiant : {{ user.username }}{% else %}Courriel : {{ user.email }}{% endif %}
-       * Mot de passe : celui que vous venez de choisir lors de l'inscription.
+* {% if user.username %}Identifiant : {{ user.username }}{% else %}Courriel : {{ user.email }}{% endif %}
+* Mot de passe : celui que vous venez de choisir lors de l'inscription.

 Pour vous connecter, veuillez cliquer sur le lien suivant :

-       {{ login_url }}
+  {{ login_url }}
 {% endautoescape %}

#18

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

Need review please.

#19

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

  • Statut changé de Nouveau à En cours

Ack.

#20

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

  • Statut changé de En cours à Résolu (à déployer)

Je l'ai poussé.

commit ad488faa428dbca79973883689410550a12f79b0
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Thu Oct 19 16:46:07 2017 +0200

    templates: add registration_success email templates (#19566)
#21

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

Merci.

#23

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

Ack.

#24

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

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

Formats disponibles : Atom PDF