HomePhabricator
Diffusion Comm f8120854b4f3

[lib] Split change thread settings spec

Description

[lib] Split change thread settings spec

Summary:
Avoid sending unnecessary data and simplify the spec logic.

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

Depends on D13052

Test Plan:
Checked two scenarios:

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

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/D13053

Details

Provenance
tomekAuthored on Aug 12 2024, 5:03 AM
Reviewer
inka
Differential Revision
D13053: [lib] Split change thread settings spec
Parents
rCOMMd2fee63e6327: [lib] Split add members spec
Branches
Unknown
Tags
Unknown