HomePhabricator
Diffusion Comm 359d8031d62d

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

Description

[lib] Use UPDATE_THREAD_READ_STATUS update to update the read status

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

Reviewers: kamil, will

Reviewed By: kamil

Subscribers: ashoat

Differential Revision: https://phab.comm.dev/D13424

Details

Provenance
tomekAuthored on Mon, Sep 23, 4:02 AM
Reviewer
kamil
Differential Revision
D13424: [lib] Use `UPDATE_THREAD_READ_STATUS` update to update the read status
Parents
rCOMM29f65d7d29a7: [terraform] upgrade blob-server on staging to 1.1.3-staging
Branches
Unknown
Tags
Unknown