Development #38054
Après une demande ré-init d'un mot de passe, ne pas revenir sur la page de login
0%
Description
... parce que ça ne sert à rien (la personne ne va pas se loger) et que le message "Si votre adresse électronique est présente dans notre base, vous recevrez un courriel contenant les instructions pour réinitialiser votre mot de passe." se retrouve peu visible dans l'affaire.
Il serait préférable d'afficher une page simple telle que celle après la demande d'inscription ; en y reprenant d'ailleurs les indications sur l'anti-spam.
Quelque chose comme :
<b>Si cette adresse électronique est présente dans notre base, un courriel a été envoyé à totonono@gmail.com.</b> <b>Suivez les instructions dans ce courriel pour choisir un nouveau mot de passe.</b> Ce message peut mettre plusieurs minutes à vous parvenir. Il peut également être considéré comme un pourriel (spam) : n'oubliez pas de regarder dans votre dossier «courriers indésirables». Si malgré cela vous n'avez pas reçu les instructions, ajoutez l'adresse ne-pas-repondre+prod@entrouvert.com à votre carnet d’adresses ou à votre liste d’expéditeurs autorisés, puis recommencez la procédure d'inscription.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Fichier 0001-views-better-display-password-reset-instructions-380.patch 0001-views-better-display-password-reset-instructions-380.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Voilà, à part que je n'ai pas mis la ligne à propos de l'adresse à ajouter, parce que c'est pas joli d'hardcoder l'email d'EO.
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Fichier 0001-views-better-display-password-reset-instructions-380.patch 0001-views-better-display-password-reset-instructions-380.patch ajouté
Avec des tests qui passent, la politesse en prime.
Mis à jour par Thomas Noël il y a plus de 4 ans
Valentin Deniaud a écrit :
Voilà, à part que je n'ai pas mis la ligne à propos de l'adresse à ajouter, parce que c'est pas joli d'hardcoder l'email d'EO.
Tu peux regarder comment c'est fait sur #37923 (ad3f27ef3cff028a1c9be7c10551f013a53ee399) : on ajoute settings.DEFAULT_FROM_EMAIL dans le contexte via un get_context_data, et hop.
Aussi, tu peux copier-coller dans ton template cette partie de src/authentic2/templates/registration/registration_complete.html :
{% blocktrans %}
The email may take several minutes to be received. It can also be
considered as spam: please look in your "junk mail" folder.
{% endblocktrans %}
afin de mutualiser le travail de traduction.
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Fichier 0001-views-better-display-password-reset-instructions-380.patch 0001-views-better-display-password-reset-instructions-380.patch ajouté
C'est fait, merci.
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Fichier 0001-views-better-display-password-reset-instructions-380.patch 0001-views-better-display-password-reset-instructions-380.patch ajouté
(vraiment fait cette fois)
Mis à jour par Thomas Noël il y a plus de 4 ans
Mmmh... mais ça marche ? J'ai l'impression que PasswordResetInstructionsView ne va pas avoir from_email_address dans son contexte...
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Fichier 0001-views-better-display-password-reset-instructions-380.patch 0001-views-better-display-password-reset-instructions-380.patch ajouté
Effectivement j'avais déjà oublié comment marchait mon propre code, 24h après.
Mis à jour par Thomas Noël il y a plus de 4 ans
Valentin Deniaud a écrit :
Effectivement j'avais déjà oublié comment marchait mon propre code, 24h après.
Attention, t'as trop copié collé :) Tu parles de "registration process" alors que non.
Et si tu as envie de faire bien, tu peux compléter un poil tes tests avec
settings.DEFAULT_FROM_EMAIL = 'show only addr <noreply@example.net>'et vérifier la présence de noreply@example dans le texte affiché (et l'absence de "show only addr").
Mis à jour par Valentin Deniaud il y a plus de 4 ans
Mis à jour par Frédéric Péters il y a plus de 4 ans
+ If your email address exists in ou database, an email has been sent to {{ email }}.
our.
+ The email may take several minutes to be received. It can also be + considered as spam: please look in your "junk mail" folder.
L'ambition de partager la traduction va se heurter à des espaces en nombre différent.
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Fichier 0001-views-better-display-password-reset-instructions-380.patch 0001-views-better-display-password-reset-instructions-380.patch ajouté
blocktrans désindentés.
Mis à jour par Frédéric Péters il y a plus de 4 ans
+ If your email address exists in ou database, an email has been sent to {{ email }}.
Plus explicitement : à la place de "ou database" il faut écrire "our database".
Mis à jour par Valentin Deniaud il y a plus de 4 ans
Mis à jour par Thomas Noël il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 044790eb91113ce81bb4d6c4faa10caac39177a4
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Tue Jan 7 15:11:57 2020 +0100
views: better display password reset instructions (#38054)Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
views: better display password reset instructions (#38054)