Mirrors/chatmail-core
1
0
Fork 0
mirror of https://github.com/chatmail/core.git synced 2026-04-17 21:46:35 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix: Emit MsgsChanged instead of MsgsNoticed on self-MDN if chat still has fresh messages

Browse Source 
Otherwise if the user reads messages being offline and then the device comes online, sent MDNs will
remove all notifications from other devices even if new messages have arrived. Notifications not
removed at all look more acceptable.
This commit is contained in:
iequidoo
2026-01-20 18:22:12 -03:00
committed by iequidoo
parent 4ccd3cb665
commit 566395f1fa
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/receive_imf.rs
View File

@@ -1017,12 +1017,12 @@ UPDATE msgs SET state=? WHERE
)
.await
.context("UPDATE msgs.state")?;
// Removes all notifications for the chat in UIs. Ideally should be under
// `if chat_id.get_fresh_msg_cnt(context).await? == 0`, but this causes a not
// updated profile badge counter in the UIs as of v2.35.0. Removed notifications for
// new messages is an already existing and known problem, so let's emit the event
// unconditionally for now.
context.emit_event(EventType::MsgsNoticed(chat_id));
if chat_id.get_fresh_msg_cnt(context).await? == 0 {
// Removes all notifications for the chat in UIs.
context.emit_event(EventType::MsgsNoticed(chat_id));
} else {
context.emit_msgs_changed_without_msg_id(chat_id);
}
chatlist_events::emit_chatlist_item_changed(context, chat_id);
}
if archived_chats_maybe_noticed {