Development #20830
password-reset: si le compte est inactif, envoyer tout de même un courriel
0%
Description
Pour indiquer à la personne qu'on ne va rien faire son compte est inactif.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Sujet changé de password-reset: si le compte est inactif, envoyez tout de même un mail à password-reset: si le compte est inactif, envoyer tout de même un courriel
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Fichier 0001-WIP-send-notification-emails-for-unauthorized-passwo.patch 0001-WIP-send-notification-emails-for-unauthorized-passwo.patch ajouté
- Patch proposed changé de Non à Oui
Pas encore écrit les templates
correspondants, mais ce serait quelque chose comme ça, non ?
Edit : Et je n'ai pas encore eu le temps de regarder comment sont testées les différentes fonctionnalités d'échange par courriel avec l'utilisateur. Ce sera l'occasion parfaite.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Oui c'est presque ça, j'ajouterai: ajouter un log au niveau INFO pour les utilisateurs inactifs parce send_templated_email() ne log rien.
Pour les tests sur les mails il y a une fixture mailoutbox qui est une liste et qui contient les derniers mails envoyés depuis le début du test, ça remplace complètement l'envoi fait via SMTP, et ça donne accès à l'objet MailMessage de Django; faire un grep mailoutbox dans les test pour le voir à l'usage.
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Fichier 0001-WIP-send-notification-emails-for-unauthorized-passwo.patch 0001-WIP-send-notification-emails-for-unauthorized-passwo.patch ajouté
Voilà pour les logs et les templates. Je me penche maintenant sur les tests.
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Fichier 0001-send-notification-emails-for-unauthorized-password-r.patch 0001-send-notification-emails-for-unauthorized-password-r.patch ajouté
Ajouté un test, mais je ne suis pas certain que ce soit la façon la plus orthodoxe de tester la complétion du PasswordResetForm
.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Lié à Development #26907: Cycle de vie des comptes ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
- Lié à Development #12376: sur "mot de passe oublié ?" permettre d'avoir un message différent quand l'email est inconnu (?) ajouté
Mis à jour par Valentin Deniaud il y a environ 3 ans
- Assigné à changé de Paul Marillonnet à Valentin Deniaud
Mis à jour par Valentin Deniaud il y a environ 3 ans
- Fichier 0001-misc-send-password-reset-email-even-if-disabled-acco.patch 0001-misc-send-password-reset-email-even-if-disabled-acco.patch ajouté
- Statut changé de Nouveau à Solution proposée
Je reprends ça, basé sur #47469 et remis au goût du jour.
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
- Planning mis à Non
Mis à jour par Valentin Deniaud il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 2c6c03a9c8fe09520edaa04dd3c7e3d28d24b203 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Mar 3 09:59:43 2021 +0100 misc: send password reset email even if disabled account (#20830)
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: send password reset email even if disabled account (#20830)