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)
Mon, Jan 27, 6:54 AM
Unknown Object (File)
Sun, Jan 26, 9:13 PM
Unknown Object (File)
Sun, Jan 26, 6:56 AM
Unknown Object (File)
Sun, Jan 26, 6:56 AM
Unknown Object (File)
Sun, Jan 26, 6:56 AM
Unknown Object (File)
Sat, Jan 25, 7:57 AM
Unknown Object (File)
Wed, Jan 22, 1:55 PM
Unknown Object (File)
Wed, Jan 22, 9:46 AM

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
No Lint Coverage
Unit
No Test Coverage