Projet

Général

Profil

Development #26910

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

Ajouté par Benjamin Dauvergne il y a plus de 5 ans. Mis à jour il y a presque 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
02 octobre 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

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


Fichiers


Demandes liées

Lié à Publik - Development #26907: Cycle de vie des comptesFermé24 novembre 2020

Actions
Lié à Authentic 2 - Development #27823: Avertir par courriel l'utilisateur lorsqu'il demande la suppression de son propre compteFermé07 novembre 2018

Actions

Révisions associées

Révision a55e3eae (diff)
Ajouté par Paul Marillonnet il y a presque 5 ans

warn user after account self-deletion (#26910)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

#2

Mis à jour par Paul Marillonnet il y a plus de 5 ans

  • Assigné à mis à Paul Marillonnet
#3

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 ?

#4

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.

#5

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é
#6

Mis à jour par Paul Marillonnet il y a plus de 5 ans

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

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.

#8

Mis à jour par Paul Marillonnet il y a plus de 5 ans

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

#9

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.

#10

Mis à jour par Paul Marillonnet il y a plus de 5 ans

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

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.

#12

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Statut changé de Solution proposée à En cours
#13

Mis à jour par Paul Marillonnet il y a environ 5 ans

#14

Mis à jour par Paul Marillonnet il y a environ 5 ans

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

#15

Mis à jour par Benjamin Dauvergne il y a environ 5 ans

  • Statut changé de Solution proposée à Solution validée

Ack.

#16

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)
#17

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

Formats disponibles : Atom PDF