Project

General

Profile

Development #38054

Après une demande ré-init d'un mot de passe, ne pas revenir sur la page de login

Added by Thomas Noël 8 months ago. Updated 6 months ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
28 Nov 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

0001-views-better-display-password-reset-instructions-380.patch View (3.85 KB) Valentin Deniaud, 07 Jan 2020 03:16 PM

0001-views-better-display-password-reset-instructions-380.patch View (6.08 KB) Valentin Deniaud, 07 Jan 2020 04:02 PM

0001-views-better-display-password-reset-instructions-380.patch View (6.57 KB) Valentin Deniaud, 08 Jan 2020 11:58 AM

0001-views-better-display-password-reset-instructions-380.patch View (6.7 KB) Valentin Deniaud, 08 Jan 2020 12:07 PM

0001-views-better-display-password-reset-instructions-380.patch View (6.67 KB) Valentin Deniaud, 08 Jan 2020 02:33 PM

0001-views-better-display-password-reset-instructions-380.patch View (7.03 KB) Valentin Deniaud, 08 Jan 2020 03:06 PM

0001-views-better-display-password-reset-instructions-380.patch View (7.01 KB) Valentin Deniaud, 08 Jan 2020 03:42 PM

0001-views-better-display-password-reset-instructions-380.patch View (7.01 KB) Valentin Deniaud, 08 Jan 2020 04:04 PM

Associated revisions

Revision abed3fa8 (diff)
Added by Valentin Deniaud 6 months ago

views: better display password reset instructions (#38054)

History

#2 Updated by Valentin Deniaud 6 months ago

  • Assignee set to Valentin Deniaud

#3 Updated by Valentin Deniaud 6 months ago

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.

#4 Updated by Valentin Deniaud 6 months ago

Avec des tests qui passent, la politesse en prime.

#5 Updated by Thomas Noël 6 months ago

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.

#8 Updated by Thomas Noël 6 months ago

Mmmh... mais ça marche ? J'ai l'impression que PasswordResetInstructionsView ne va pas avoir from_email_address dans son contexte...

#9 Updated by Valentin Deniaud 6 months ago

Effectivement j'avais déjà oublié comment marchait mon propre code, 24h après.

#10 Updated by Thomas Noël 6 months ago

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 &lt;&gt;'

et vérifier la présence de noreply@example dans le texte affiché (et l'absence de "show only addr").

#12 Updated by Frédéric Péters 6 months ago

+      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.

#14 Updated by Frédéric Péters 6 months ago

+     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".

#16 Updated by Thomas Noël 6 months ago

  • Status changed from Solution proposée to Solution validée

#17 Updated by Valentin Deniaud 6 months ago

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

#18 Updated by Frédéric Péters 6 months ago

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

Also available in: Atom PDF