Skip to content

Conversation

iequidoo
Copy link
Collaborator

@iequidoo iequidoo commented Jan 8, 2025

See commit message.

#6351 (comment) says:

FTR, if we have an actual user value for emitting MsgsNoticed on receipt of an IMAP-seen message, and test that MsgsNoticed is not emitted for every reaction and mdn, then I'm not against it.

This PR avoids emitting MsgsNoticed for the trash chat and adds the corresponding tests for reactions and MDNs.

@iequidoo iequidoo marked this pull request as ready for review January 8, 2025 19:27
@iequidoo iequidoo requested a review from Hocuri January 8, 2025 19:28
@iequidoo iequidoo force-pushed the iequidoo/MsgsNoticed-on-seen-msg branch from d0b64a4 to d4bd239 Compare August 19, 2025 20:18
@iequidoo iequidoo marked this pull request as draft August 19, 2025 20:22
@iequidoo iequidoo force-pushed the iequidoo/MsgsNoticed-on-seen-msg branch from d4bd239 to 9381e8d Compare August 20, 2025 11:45
@iequidoo iequidoo marked this pull request as ready for review August 20, 2025 11:55
`imap::Session::sync_seen_flags()` emits `MsgsNoticed` for existing messages seen on other devices,
so `receive_imf` should do the same when it receives a seen message. Otherwise a multi-device user
may see a new message notification on device A, just swipe it, then see another new message
notification and mark it as read, and when a device B goes online, it will show a notification for
the first message, and it won't be removed because `MsgsNoticed` isn't emitted. I have checked this
with my DC Android and Desktop. With this fix the notification can be removed if UIs handle
`MsgsNoticed` accordingly, though currently i can't achieve this behaviour in Desktop.

Also add blocked chat's messages and silent group changes messages as `InNoticed` instead of
`InSeen`, they don't need to cause `MsgsNoticed` events. This doesn't change any UX, checked this in
Desktop.
@iequidoo iequidoo force-pushed the iequidoo/MsgsNoticed-on-seen-msg branch from 9381e8d to 313a200 Compare August 31, 2025 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant