Situphen
commented
9 years ago Renvoi d'une notification si on note une réponse comme "non-lue"
Attention à ne pas renvoyer de courriel
Open ChantyTaguan opened 9 years ago
Situphen
commented
9 years ago Renvoi d'une notification si on note une réponse comme "non-lue"
Attention à ne pas renvoyer de courriel
ChantyTaguan
commented
9 years ago | TO DO | Status | Test | Doc | Commentaire |
|---|---|---|---|---|
| Abonnement à la création | OK | OK | OK | / |
| Abonnement sur réponse | OK | OK | OK | / |
| Envoi des notifications lors d'une réponse au topic | OK | OK | OK | / |
| Envoi d'un email pour les abonnements avec email | OK | OK | OK | / |
| Abonnement/désabonnement via la sidebar | OK | OK | OK | / |
| Renvoi d'une notification si on note une réponse comme "non-lue" | OK | OK | OK | / |
Concernant les actions qu'on peut faire dans la sidebar, je me demande si d'un point de vue performance ça vaudrait pas la peine de loader les abonnements du user au topic dans la vue même.
Parce que là on utilise les filtres pour chaque actions possible, et parfois plusieurs fois par actions. Et chaque filtre fait une requête pour trouver l'abonnement, qui est parois le même concerné pour plusieurs actions. Par exemple pour ce type d'abonnement-ci (réponse au topic), on utilise 8 fois le filtre dans le template alors qu'on pourrait simplement charger une fois l'abonnement.
Pour la colonne "Doc", j'ai considéré que c'était OK si les méthodes/modèles/vues se rapportant à l'action étaient documentées (ou tellement claires que c'est pas nécessaire). Je suppose qu'il faudra mettre la doc technique read the doc à jour aussi mais autant faire un ticket spécial pour ça.
Abonnement : Topic Notification : Post Type : NEW_CONTENT
Lien vers : premier message non lu du topic