Project

General

Profile

Development #26910

La suppression d'un utilisateur par lui même doit provoquer l'envoi d'un mail de notification

Added by Benjamin Dauvergne 10 months ago. Updated 4 months ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
02 Oct 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Template: authentic2/account_delete_notification_<ou.slug>.html authentic2/account_delete_notification.html

0001-WIP-warn-user-after-account-self-deletion-26910.patch View (5.5 KB) Paul Marillonnet, 15 Nov 2018 09:35 AM

0001-warn-user-after-account-self-deletion-26910.patch View (5.52 KB) Paul Marillonnet, 15 Nov 2018 10:06 AM

0001-warn-user-after-account-self-deletion-26910.patch View (4.94 KB) Paul Marillonnet, 15 Nov 2018 10:52 AM

0001-warn-user-after-account-self-deletion-26910.patch View (5.07 KB) Paul Marillonnet, 22 Mar 2019 04:16 PM

0001-warn-user-after-account-self-deletion-26910.patch View (5.08 KB) Paul Marillonnet, 22 Mar 2019 05:38 PM


Related issues

Related to Publik - Development #26907: Cycle de vie des comptes Nouveau 02 Oct 2018
Related to Authentic 2 - Development #27823: Avertir par courriel l'utilisateur lorsqu'il demande la suppression de son propre compte En cours 07 Nov 2018

Associated revisions

Revision a55e3eae (diff)
Added by Paul Marillonnet 4 months ago

warn user after account self-deletion (#26910)

History

#1 Updated by Benjamin Dauvergne 10 months ago

#2 Updated by Paul Marillonnet 10 months ago

  • Assignee set to Paul Marillonnet

#3 Updated by Paul Marillonnet 9 months ago

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 ?

#4 Updated by Benjamin Dauvergne 9 months ago

La première réponse. La deuxième est intéressante, mais ça fera l'objet d'un autre ticket.

#5 Updated by Paul Marillonnet 9 months ago

  • Related to Development #27823: Avertir par courriel l'utilisateur lorsqu'il demande la suppression de son propre compte added

#6 Updated by Paul Marillonnet 8 months ago

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 ?

#7 Updated by Frédéric Péters 8 months ago

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.

#8 Updated by Paul Marillonnet 8 months ago

Ok, merci.
Je verrais donc bien le code ainsi.

#9 Updated by Benjamin Dauvergne 8 months ago

send_account_deletion_mail devrait prendre comme seuls paramètres, request et user, il devrait se débrouiller pour définir le context ensuite.

#10 Updated by Paul Marillonnet 8 months ago

Benjamin Dauvergne a écrit :

send_account_deletion_mail devrait prendre comme seuls paramètres, request et user, 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é.

#11 Updated by Benjamin Dauvergne 4 months ago

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.

#12 Updated by Benjamin Dauvergne 4 months ago

  • Status changed from Solution proposée to En cours

#13 Updated by Paul Marillonnet 4 months ago

#14 Updated by Paul Marillonnet 4 months ago

Corrigé une typo dans le message envoyé à l'usager.

#15 Updated by Benjamin Dauvergne 4 months ago

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

Ack.

#16 Updated by Paul Marillonnet 4 months ago

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

#17 Updated by Frédéric Péters 4 months ago

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

Also available in: Atom PDF