Produits Entr'ouvert » Authentic 2

Bon début. Ceci dit en réalité l'idée c'est aussi de supprimer les mentions du genre :

Since your account was last active on {{ last_login_date }}

qui sont sujettes à trop de (mauvaises) interprétations dans les unused_account_delete*.

Et aussi supprimer la date dans unused_account_alert_subject.txt, dire juste "Votre compte est inactif et va bientôt être supprimé" (ce qui ajoute une information-alerte importante au passage, d'ailleurs).

Thomas Noël a écrit :

Bon début. Ceci dit en réalité l'idée c'est aussi de supprimer les mentions du genre :

Since your account was last active on {{ last_login_date }}

qui sont sujettes à trop de (mauvaises) interprétations dans les unused_account_delete*.

Oui ok, j’avais loupé ce point qui pourtant était déjà en description de ticket. C’est corrigé dans cette version, qui par ailleurs insiste sur le fait que le compte va être supprimé.
J’en profite aussi pour corriger une phrase d’un anglais un peu trop approximatif (je trouve) en fin de courriel d’alerte.

Reste encore un bout sur src/authentic2/templates/authentic2/unused_account_alert_subject.txt :)

Et sur le test test_clean_unused_account_human_duration_format ajouter quand même des assert qui vérifient un peu qu'un bout de phrase est là (pas le formatted, mais autre chose).

Thomas Noël a écrit :

Reste encore un bout sur src/authentic2/templates/authentic2/unused_account_alert_subject.txt :)

Ouch, excès de vitesse à l’écriture du patch, mes excuses.
C’est corrigé, et j’en ai aussi profité pour retirer le chargement de la bibliothèque de balises et filtres humanize dans tous ces gabarits, elle ne sert maintenant plus à rien.

Et sur le test test_clean_unused_account_human_duration_format ajouter quand même des assert qui vérifient un peu qu'un bout de phrase est là (pas le formatted, mais autre chose).

J’ai complété un peu le test qui était trop léger en effet, avec notamment des valeurs de délai entre alerte et suppression réalistes.

Thomas Noël a écrit :

Détail, mais tu peux factoriser les traductions dans unused_account_delete_body.html et unused_account_delete_body.txt

Oui ok, les chaînes internationalisées sont maintenant les mêmes dans ces deux gabarits.

commit 575cf199e6191b321b3b707778e17c8ca53f9206
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Mon Feb 6 09:11:25 2023 +0100

    custom_user: provide more generic user-inactivity notifications (#74178)

    Authentic now exposes a keepalive API endpoint, meaning that the user's
    last activity event was not necessarily a user login event.