HomePhabricator
Diffusion Comm e0ba6e60f3f0

[lib] Check the timestamps when changing thread settings

Description

[lib] Check the timestamps when changing thread settings

Summary:
Only update properties when they are older than the operation that is being processed.

https://linear.app/comm/issue/ENG-9118/update-changethreadsettingsspec

Depends on D13214

Test Plan:
Performed 3 operations in a row:

  1. Thread creation with timestamp T
  2. Thread name change with timestamp T + 2
  3. Thread name and color change with timestamp T + 1

Made sure that thread color matched the value from 3 and name matched the value from 2.

Reviewers: kamil, inka

Reviewed By: inka

Subscribers: ashoat

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

Details

Provenance
tomekAuthored on Aug 30 2024, 9:59 AM
Reviewer
inka
Differential Revision
D13215: [lib] Check the timestamps when changing thread settings
Parents
rCOMMf72ecb2ea151: [lib] Introduce thread timestamps on the JS side
Branches
Unknown
Tags
Unknown