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
Paths
| Differential D13233 Authored by tomek on Sep 3 2024, 7:17 AM.
Details 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
Event TimelineHerald added a subscriber: ashoat. · View Herald TranscriptSep 3 2024, 7:17 AM2024-09-03 07:17:02 (UTC-7) Harbormaster completed remote builds in B31411: Diff 43857.Sep 3 2024, 7:48 AM2024-09-03 07:48:43 (UTC-7) This revision is now accepted and ready to land.Sep 5 2024, 12:56 AM2024-09-05 00:56:18 (UTC-7) Closed by commit rCOMMff656da6fda9: [lib] Check the timestamps when removing thread members (authored by tomek). · Explain WhySep 6 2024, 8:10 AM2024-09-06 08:10:50 (UTC-7) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 43937 lib/shared/dm-ops/remove-members-spec.js
|