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)
Sat, Jul 6, 3:37 PM
Unknown Object (File)
Sat, Jul 6, 8:56 AM
Unknown Object (File)
Tue, Jul 2, 6:48 PM
Unknown Object (File)
Mon, Jul 1, 1:41 PM
Unknown Object (File)
Sun, Jun 30, 5:58 PM
Unknown Object (File)
Sat, Jun 29, 9:04 PM
Unknown Object (File)
Fri, Jun 28, 3:00 PM
Unknown Object (File)
Wed, Jun 26, 4:00 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