Development #26910
La suppression d'un utilisateur par lui même doit provoquer l'envoi d'un mail de notification
0%
Description
Template: authentic2/account_delete_notification_<ou.slug>.html authentic2/account_delete_notification.html
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Lié à Development #26907: Cycle de vie des comptes ajouté
Mis à jour par Paul Marillonnet il y a plus de 5 ans
On parle d'une notification a posteriori, càd l'informant que son compte a été supprimé ?
Ou bien un message d'avertissement, avec peut-être une URL opaque pour confirmer la suppression de son compte ?
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
La première réponse. La deuxième est intéressante, mais ça fera l'objet d'un autre ticket.
Mis à jour par Paul Marillonnet il y a plus de 5 ans
- Lié à Development #27823: Avertir par courriel l'utilisateur lorsqu'il demande la suppression de son propre compte ajouté
Mis à jour par Paul Marillonnet il y a plus de 5 ans
- Fichier 0001-WIP-warn-user-after-account-self-deletion-26910.patch 0001-WIP-warn-user-after-account-self-deletion-26910.patch ajouté
- Statut changé de Nouveau à En cours
Une première version, qui reprend le format en <nom du template>_{subject.txt,body.{txt,html}}
.
J'ai préfixé le message de commit par WIP parce qu'il manque encore l'utilisation d'un template spécifique à l'OU.
Question au reviewer : Est-ce qu'on teste l'existence des fichiers de templates spécifiques à l'OU ? Ou bien est-ce qu'on tente l'envoi de toute façon ? Est-ce que dans ce cas on bascule sur les templates par défaut si cet envoi échoue ?
Mis à jour par Frédéric Péters il y a plus de 5 ans
Question au reviewer : Est-ce qu'on teste l'existence des fichiers de templates spécifiques à l'OU ? Ou bien est-ce qu'on tente l'envoi de toute façon ? Est-ce que dans ce cas on bascule sur les templates par défaut si cet envoi échoue ?
De manière générale pour ce genre de choses tu dois construire la liste des templates potentiels, genre ["/plop/%s/toto.txt" % ou, "/plop/toto.txt"], passer ça à django et lui laisser prendre le premier qu'il trouve; pas de gestion manuelle de recherche de template.
Mis à jour par Paul Marillonnet il y a plus de 5 ans
- Fichier 0001-warn-user-after-account-self-deletion-26910.patch 0001-warn-user-after-account-self-deletion-26910.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Ok, merci.
Je verrais donc bien le code ainsi.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
send_account_deletion_mail
devrait prendre comme seuls paramètres, request
et user
, il devrait se débrouiller pour définir le context ensuite.
Mis à jour par Paul Marillonnet il y a plus de 5 ans
- Fichier 0001-warn-user-after-account-self-deletion-26910.patch 0001-warn-user-after-account-self-deletion-26910.patch ajouté
Benjamin Dauvergne a écrit :
send_account_deletion_mail
devrait prendre comme seuls paramètres,request
etuser
, il devrait se débrouiller pour définir le context ensuite.
Ok. Dans le code de la fonction appelante ça donne un disgracieux send_..._mail(self.request, self.request.user)
, mais c'est vrai qu'on peut imaginer utiliser cette fonction à l'avenir pour un autre utilisateur que l'utilisateur authentifié.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
Je changerai le template :
Your account deletion request on {{ site }} has been received.
pour
Your account on {{ site }} has been deleted. All related datas will be deleted today. You cannot login with it anymore.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution proposée à En cours
Mis à jour par Paul Marillonnet il y a environ 5 ans
- Fichier 0001-warn-user-after-account-self-deletion-26910.patch 0001-warn-user-after-account-self-deletion-26910.patch ajouté
- Statut changé de En cours à Solution proposée
Mis à jour par Paul Marillonnet il y a environ 5 ans
- Fichier 0001-warn-user-after-account-self-deletion-26910.patch 0001-warn-user-after-account-self-deletion-26910.patch ajouté
Corrigé une typo dans le message envoyé à l'usager.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution proposée à Solution validée
Ack.
Mis à jour par Paul Marillonnet il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit a55e3eae3b6d1ef5af0eb2df54dc8897316729f6 Author: Paul Marillonnet <pmarillonnet@entrouvert.com> Date: Wed Nov 14 11:04:13 2018 +0100 warn user after account self-deletion (#26910)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
warn user after account self-deletion (#26910)