entrouvert
/
authentic
16
0
Fork 0
Code Issues Pull Requests 35 Releases Activity

user_details: use model's effective alert date when relevant (#75255) #123

Merged
pmarillonnet merged 1 commits from wip/75255-user-details-effective-deletion-alert-date into main 2024-01-16 13:52:24 +01:00
Conversation 0 Commits 1 Files Changed 2 +36 -17
pmarillonnet commented 2023-09-11 15:23:11 +02:00
Owner
Copy Link
No description provided.
pmarillonnet added 1 commit 2023-09-11 15:23:12 +02:00
bdauvergne requested changes 2023-12-21 11:47:56 +01:00
src/authentic2/manager/templates/authentic2/manager/user_detail.html Outdated
@ -103,0 +89,4 @@
{% if object.last_account_deletion_alert or alert_date %}
<p class="a2-manager-user-date-alert">
{% if object.last_account_deletion_alert %}
{% blocktrans %}Deletion alert email sent on {{ alert_date }}.{% endblocktrans %}
bdauvergne commented 2023-12-21 11:47:42 +01:00
Owner
Copy Link

Pourquoi ne pas afficher directement la date object.last_account_deletion_alert ici ? Il me semble que c'est l'objet du ticket, alert_date continue à être la date estimée.

Pourquoi ne pas afficher directement la date object.last_account_deletion_alert ici ? Il me semble que c'est l'objet du ticket, `alert_date` continue à être la date estimée.
pmarillonnet commented 2024-01-03 14:37:49 +01:00
Author
Owner
Copy Link

Oups, c’est précisément l’objet du ticket oui, dommage :)
C’est corrigé dans cette nouvelle version de la PR.

Oups, c’est précisément l’objet du ticket oui, dommage :) C’est corrigé dans cette nouvelle version de la PR.
pmarillonnet commented 2024-01-03 14:48:24 +01:00
Author
Owner
Copy Link

Et notamment dans cette nouvelle version, on affiche aussi l’heure précise d’envoi de l’alerte en plus du jour, je pense que ça peut être pertinent pour du support de niveau 1 par les administrateurs fonctionnels (e.g. pouvoir dire à un usager à quelle heure précisément il a ou aurait dû recevoir l’alerte).

Et notamment dans cette nouvelle version, on affiche aussi l’heure précise d’envoi de l’alerte en plus du jour, je pense que ça peut être pertinent pour du support de niveau 1 par les administrateurs fonctionnels (e.g. pouvoir dire à un usager à quelle heure précisément il a ou aurait dû recevoir l’alerte).
pmarillonnet force-pushed wip/75255-user-details-effective-deletion-alert-date from 86bf4718a7 to 55350f2b7f 2024-01-03 12:02:46 +01:00 Compare
pmarillonnet force-pushed wip/75255-user-details-effective-deletion-alert-date from 55350f2b7f to ee93cb94f7 2024-01-03 12:07:20 +01:00 Compare
pmarillonnet force-pushed wip/75255-user-details-effective-deletion-alert-date from ee93cb94f7 to bff7193799 2024-01-03 12:12:24 +01:00 Compare
pmarillonnet force-pushed wip/75255-user-details-effective-deletion-alert-date from bff7193799 to fe6518f714 2024-01-03 12:18:03 +01:00 Compare
pmarillonnet force-pushed wip/75255-user-details-effective-deletion-alert-date from fe6518f714 to 70169c116a 2024-01-03 14:32:13 +01:00 Compare
pmarillonnet requested review from bdauvergne 2024-01-03 14:37:53 +01:00
pmarillonnet force-pushed wip/75255-user-details-effective-deletion-alert-date from 70169c116a to 8e8c7080ec 2024-01-03 14:46:18 +01:00 Compare
bdauvergne requested changes 2024-01-16 13:02:42 +01:00
src/authentic2/manager/templates/authentic2/manager/user_detail.html Outdated
@ -103,0 +89,4 @@
{% if alert_date %}
<p class="a2-manager-user-date-alert">
{% if object.last_account_deletion_alert %}
{% blocktrans with alert_date=object.last_account_deletion_alert %}Deletion alert email sent on {{ alert_date }}.{% endblocktrans %}
bdauvergne commented 2024-01-16 13:02:38 +01:00
Owner
Copy Link

Pas compris, tu affiches toujours alert_date ici.

Pas compris, tu affiches toujours alert_date ici.
bdauvergne commented 2024-01-16 13:03:18 +01:00
Owner
Copy Link

Mal lu c'est dans le blocktrans que l'alias est défini.

Mal lu c'est dans le blocktrans que l'alias est défini.
bdauvergne marked this conversation as resolved
bdauvergne approved these changes 2024-01-16 13:03:38 +01:00
pmarillonnet force-pushed wip/75255-user-details-effective-deletion-alert-date from 8e8c7080ec to 119ae53ce3 2024-01-16 13:46:01 +01:00 Compare
pmarillonnet merged commit 119ae53ce3 into main 2024-01-16 13:52:24 +01:00
pmarillonnet deleted branch wip/75255-user-details-effective-deletion-alert-date 2024-01-16 13:52:24 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
bdauvergne
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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/authentic#123
No description provided.
Delete Branch "wip/75255-user-details-effective-deletion-alert-date"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?