templates: redo account deletion emails (#78467) #312

fpeters · 2023-06-13T18:00:58+02:00

fpeters commented

2023-06-13 18:00:58 +02:00

No description provided.

fpeters added 1 commit 2023-06-13 18:00:59 +02:00

gitea/publik-base-theme/pipeline/head This commit looks good Details

c10b27a5ea templates: redo account deletion emails (#78467 )

tnoel requested changes 2023-06-13 18:37:40 +02:00

templates/authentic2/account_delete_notification_body.html

						
				@ -5,3 +5,1 @@

				  <p>Votre compte sur le site {{site}} a été supprimé.</p>

				  <p>Toutes les données associées seront supprimées aujourd’hui.</p>

				  <p>Vous ne pouvez dès à présent plus vous connecter.</p>

				  <p>Votre compte sur le site {{site}} a été supprimé, conformément à votre demande.</p>

tnoel commented

2023-06-13 18:36:46 +02:00

Absolument pas sûr du "conformément à votre demande", je pense que ce message est également envoyé en cas de suppression automatique du compte. Voire même d'autres cas...

fpeters commented

2023-06-13 18:52:31 +02:00

J'étais prêt à faire cette remarque dans le ticket mais j'ai vérifié et ça ne m'a pas semblé être le cas :

def send_account_deletion_mail(request, user):
    """Send an account deletion notification mail to a user.

    Can raise an smtplib.SMTPException
    """
    logger = logging.getLogger(__name__)
    context = {'full_name': user.get_full_name(), 'user': user, 'site': request.get_host()}
    template_names = ['authentic2/account_delete_notification']

et seul endroit où cette fonction est appelée, la vue

class ValidateDeletionView(TemplateView):
...
    def delete_account(self, request):
        utils_misc.send_account_deletion_mail(self.request, self.user)

J'étais prêt à faire cette remarque dans le ticket mais j'ai vérifié et ça ne m'a pas semblé être le cas : ``` def send_account_deletion_mail(request, user): """Send an account deletion notification mail to a user. Can raise an smtplib.SMTPException """ logger = logging.getLogger(__name__) context = {'full_name': user.get_full_name(), 'user': user, 'site': request.get_host()} template_names = ['authentic2/account_delete_notification'] ``` et seul endroit où cette fonction est appelée, la vue ``` class ValidateDeletionView(TemplateView): ... def delete_account(self, request): utils_misc.send_account_deletion_mail(self.request, self.user) ```

tnoel marked this conversation as resolved

fpeters requested review from tnoel 2023-06-13 18:52:35 +02:00

tnoel approved these changes 2023-06-14 01:32:37 +02:00

fpeters merged commit f6716368a8 into main

2023-06-15 09:08:36 +02:00

fpeters deleted branch wip/78467-deletion-messages

2023-06-15 09:08:36 +02:00

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-base-theme#312