Page MenuHomePhabricator
Differential D13233

[lib] Check the timestamps when removing thread members
ClosedPublic
Actions

Authored by tomek on Tue, Sep 3, 7:17 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 17, 10:58 AM
Unknown Object (File)
Tue, Sep 17, 9:59 AM
Unknown Object (File)
Mon, Sep 16, 3:18 PM
Unknown Object (File)
Mon, Sep 16, 5:43 AM
Unknown Object (File)
Mon, Sep 16, 12:09 AM
Unknown Object (File)
Sun, Sep 15, 2:29 AM
Unknown Object (File)
Sat, Sep 14, 11:55 PM
Unknown Object (File)
Sat, Sep 14, 2:24 PM
View All 20 Files
Subscribers
ashoat

Details

Reviewers
kamil
inka
Commits
rCOMMff656da6fda9: [lib] Check the timestamps when removing thread members
Summary

Check the timestamp and don't remove members when the timestamp is more recent than the operation.

https://linear.app/comm/issue/ENG-9117/update-removemembersspec

Depends on D13220

Test Plan

Perform a couple of remove and add members operations and check if the membership and timestamps are correct.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
lib/
shared/
dm-ops/
27 lines

Diff 43937

View Options

lib/shared/dm-ops/remove-members-spec.js

Loading...