Development #17109
notifications : affichage synchrone
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
22 juin 2017
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
J'ai relu le ticket où on implémente le système de notifications (#13812) sans trouver de raison au chargement asynchrone de la cellule. Étant donné que c'est entièrement construit à base de données locales, je la rendrais bien synchrone.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 7 ans
- Fichier 0001-notifications-turn-notification-cell-synchronous-171.patch 0001-notifications-turn-notification-cell-synchronous-171.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a presque 7 ans
l'asynchrone c'est pour avoir le rafraichissement, ça me semble important pour les notifications
Mis à jour par Frédéric Péters il y a presque 7 ans
Le rafraichissement régulier est indépendant, contrôlé par l'attribut ajax_refresh (qui n'est pas posé pour cette cellule).
Ça serait genre :
--- a/combo/apps/notifications/models.py
+++ b/combo/apps/notifications/models.py
@@ -108,6 +108,7 @@ class Notification(models.Model):
@register_cell_class
class NotificationsCell(CellBase):
user_dependant = True
+ ajax_refresh = 10
class Meta:
verbose_name = _('User Notifications')
Mais c'est indépendant du rendu initial.
Mis à jour par Thomas Noël il y a presque 7 ans
Donc ack. Ca me va de mettre le «ajax_refresh» dans le même patch, (turn notification cell synchronous but with refresh)
Mis à jour par Frédéric Péters il y a presque 7 ans
- Statut changé de En cours à Résolu (à déployer)
commit 71f0dd7b142db2009727a3a41c1eaf21a614acb1
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Thu Jun 22 14:26:58 2017 +0200
notifications: turn notification cell synchronous (#17109)
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: turn notification cell synchronous (#17109)