Development #25186
notifications : toujours pas bon pour avoir une suppression contrôlée par l'usager
0%
Description
J'ai lâché l'affaire dans #22732 et j'aurais du être plus attentif parce qu'Emmanuel notait bien sur la fin que le ack() devenait identique au forget(). Et la conséquence pratique c'est évidemment de ne pas pouvoir faire la différence entre la notification ackée, qu'on veut continuer à afficher (à GNM) et la notification qu'on veut voir disparaitre (après un appel à forget, à GNM).
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 6 ans
- Fichier 0001-notifications-don-t-make-ack-imply-forget-25186.patch 0001-notifications-don-t-make-ack-imply-forget-25186.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Des verbes simples : ack → ça met juste acked à True; forget → ça fait ack + ça met une date d'expiration dans le passé.
Ça va faire le job pour GNM qui est quand même le seul endroit où ce code est utilisé.
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
Ok pour l'essentiel mais au passage, comme dit dans #22732, il y a un bout de code qui plantera sur une notification sans expiration :
@property
def visible(self):
return self.end_timestamp > now()
Erreur peut-être pas visible parce ce que code mort, mais bon allons jusqu'au bout en corrigeant ou supprimant IMHO.
Mis à jour par Frédéric Péters il y a presque 6 ans
- Fichier 0001-notifications-don-t-make-ack-imply-forget-25186.patch 0001-notifications-don-t-make-ack-imply-forget-25186.patch ajouté
Yep, simplement dégager cette propriété qui n'est pas utilisée (vérifié combo et publik-base-theme).
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Statut changé de Solution proposée à Solution validée
wunderbar.
Mis à jour par Frédéric Péters il y a presque 6 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit e98519095297936ea2511d2923cd23c83862d4c4
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Jul 10 14:46:10 2018 +0200
notifications: don't make ack() imply forget() (#25186)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
notifications: don't make ack() imply forget() (#25186)