Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notifications persistantes hors fil de discussion #1101

Open
Delphgateau opened this issue Oct 11, 2024 · 2 comments
Open

Notifications persistantes hors fil de discussion #1101

Delphgateau opened this issue Oct 11, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@Delphgateau
Copy link
Collaborator

Il semblerait qu'il y ait des notifications persistantes sur la 2.8.0, bien qu'aucun fil de discussions n'aient été créés depuis une session web sur les salons concernés.
Bug remonté sur ce salon: https://app-v4.prod.lab.tchap.incubateur.net/index.html#/room/!fqnIzvoUUENjqmvGPq:agent.dinum.tchap.gouv.fr

@Delphgateau Delphgateau added the bug Something isn't working label Oct 11, 2024
@NicolasBuquet
Copy link
Contributor

La pastille de notification non-lu est affiché si le roomSummary.localUnreadEventCount de cette room est différent de 0.
->

return (roomSummary.localUnreadEventCount != 0);

Note dans le code source a propos de cette propriété :

/**
 The number of unread events wrote in the store which have their type listed in the MXSession.unreadEventType.

 @discussion: The returned count is relative to the local storage. The actual unread messages
 for a room may be higher than the returned value.
 * 

Si la pastille est affichée, elel va chercher sa valeur dans la propriété roomSummary.notificationCount.
->

return self.roomSummary.notificationCount;

Note dans le code source a propos de cette propriété :

/**
 The number of unread messages that match the push notification rules.
 It is based on the notificationCount field in /sync response.
 */

La valeur du nombre de messages non-lus d'un salon est donc déterminé et renvoyé par le backend.
Par contre, il semble traité différemment en fonction du client. Sur un même compte avec une connexion iOS et une connexion web simultanées, l'affichage des non-lus est différent :

Screenshot 2024-10-14 at 11 55 49

@NicolasBuquet
Copy link
Contributor

Réactiver le code de gestion des threads testé sur iOS ne change rien à l'affichage. (#878 )

Consulter les threads d'un salon contenant des threads permet de faire baisser le nombre de messages non-lu. Ces messages de threads sont donc bien pris en compte :

Screenshot 2024-10-14 at 11 56 03 Screenshot 2024-10-14 at 11 56 51

Les non-lus du salon peuvent alors être marqués comme lus :

Simulator Screenshot - iPhone 15 - 2024-10-14 at 11 57 26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants