Page MenuHomePhabricator

[lib] Use `UPDATE_THREAD_READ_STATUS` update to update the read status
ClosedPublic

Authored by tomek on Sep 23 2024, 4:02 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 1, 7:37 AM
Unknown Object (File)
Sun, Mar 16, 12:43 AM
Unknown Object (File)
Mar 1 2025, 4:06 AM
Unknown Object (File)
Feb 18 2025, 3:26 AM
Unknown Object (File)
Jan 15 2025, 9:33 AM
Unknown Object (File)
Jan 5 2025, 4:51 PM
Unknown Object (File)
Jan 1 2025, 6:41 AM
Unknown Object (File)
Jan 1 2025, 6:41 AM
Subscribers

Details

Summary

This update is more convenient because it doesn't override the whole thread, which fixes a bug where the unread status operation sometimes swallows other operations. For the thick threads, this update should also bump a timestamp.

https://linear.app/comm/issue/ENG-9343/updating-thread-unread-status-sometimes-swallows-queued-operations

Test Plan
  1. Send a message as one user and check if a thread of the other used got marked as unread
  2. Send a message as a user and check if a thread doesn't get marked as unread on another device of the same user
  3. Check if marking as unread results in a thread being marked as unread on another device of the same user

Diff Detail

Repository
rCOMM Comm
Branch
improvements2
Lint
No Lint Coverage
Unit
No Test Coverage