Page MenuHomePhabricator

[server] [fix] Unify permissions for `OPEN_SUBTHREAD` and `SECRET_SUBTHREAD`
ClosedPublic

Authored by jacek on May 18 2022, 3:34 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Sep 26, 11:10 PM
Unknown Object (File)
Sep 2 2024, 7:16 PM
Unknown Object (File)
Aug 17 2024, 9:34 PM
Unknown Object (File)
Aug 15 2024, 10:28 AM
Unknown Object (File)
Aug 15 2024, 10:28 AM
Unknown Object (File)
Aug 15 2024, 10:28 AM
Unknown Object (File)
Aug 15 2024, 10:28 AM
Unknown Object (File)
Aug 8 2024, 5:48 PM

Details

Summary

Unify permissions for community OPEN and SECRET subthread to solve the issue:
https://linear.app/comm/issue/ENG-1004/error-when-adding-user-that-is-not-in-parent-thread-to-open-subchannel

The diff requires database migration to fix permissions in already existing threads. It will be introduced in follow-up diff.

Test Plan

Create new Open thread nested in Secret thread - the issue described in ENG-1004 should no longer exist.
Users not in parent thread can now be added to child thread. Tested in mobile and web app.
After database migration the issue should also disappear in all already existing threads.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable