search

sucresware / 4sucres

Parce qu'à 2 on était pas assez.
https://4sucres.org
9 stars 4 forks source link

Ne pas envoyer de double notification #22

Open mgkprod opened 5 years ago

mgkprod commented 5 years ago

Par exemple, en cas de mention + nouveau message dans une discussion, on reçoit de notifications pour le même contexte, mais qui ne portent pas la même information.

Je suis d'avis de laisser le comportement actuel, parce que c'est chiant à modifier, les deux notifications ne sont pas envoyées du tout au même endroit.

innocenzi commented 5 years ago

Je comprends que ce soit compliqué, cela dit d'un point de vue logique et pratique il est préférable de s'en débarrasser.

De la même manière que la vérification ci-dessous s'assure que l'utilisateur n'a pas déjà de notification non lue pour la discussion, est-ce qu'il est possible de vérifier qu'une double notification ne va pas s'envoyer ?

https://github.com/4sucres/board/blob/8b80e14813922167310e6629fa6f702e760758e1/app/Models/Discussion.php#L124-L137

Le deux cas à vérifier auxquels je pense sont le fait qu'il n'y a pas de notification de mention ou citation avant d'envoyer une notification de nouvelle réponse, ou le fait qu'il n'y ait pas de notification de mention avant d'envoyer la notification de citation et vice-versa.

Si ça s'avère trop compliqué, il pourrait être judicieux de créer une NotificationFactory qui effectuerait les vérifications et enverrait les notifications au même endroit.