HomePhabricator
Diffusion Comm d2fee63e6327

[lib] Split add members spec

Description

[lib] Split add members spec

Summary:
Split the spec so that we don't send extraneous data to all the existing members.

https://linear.app/comm/issue/ENG-8929/split-add-members-operation

Test Plan:
Checked two scenarios:

  1. Viewer creates a thread and adds a new user to it using add_members operation
  2. Viewer is added to a thread using add_viewer_to_thread_members

In both cases the thread appeared in the store with correct membership array.

Reviewers: kamil, inka

Reviewed By: inka

Subscribers: ashoat

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

Details

Provenance
tomekAuthored on Aug 12 2024, 1:51 AM
Reviewer
inka
Differential Revision
D13052: [lib] Split add members spec
Parents
rCOMM7aa01dd12d5e: [keyserver] Recalculate permissions and reintroduce ADD_MEMBERS permission
Branches
Unknown
Tags
Unknown