Page MenuHomePhabricator

[keyserver/web] Enable announcement subchannel creation
ClosedPublic

Authored by rohan on Oct 13 2023, 12:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 1, 9:55 AM
Unknown Object (File)
Sat, Sep 28, 5:06 PM
Unknown Object (File)
Sat, Sep 28, 5:06 PM
Unknown Object (File)
Sat, Sep 28, 5:05 PM
Unknown Object (File)
Sat, Sep 28, 4:57 PM
Unknown Object (File)
Fri, Sep 27, 5:46 AM
Unknown Object (File)
Mon, Sep 23, 11:17 AM
Unknown Object (File)
Mon, Sep 23, 2:25 AM
Subscribers

Details

Summary

While working on ENG-5182, I noticed that we prevent subchannel creation to be announcement channels. From an initial investigation, it looks like it's been disabled since D5211. An error is thrown when attempting to create a new subchannel with the announcement setting because it fails the input validation.

Test Plan

Ran yarn workspace lib test (to make sure thread-responders.test.js still worked, and more importantly manually tested creating both an open announcement subchannel and a secret announcement subchannel. The permissions like I mentioned are kind of weird though since non-members of the new secret subchannel can discover it in the navigation drawer, so I'll make it part of my overall permissions fixes for ENG-5182.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

rohan requested review of this revision.Oct 13 2023, 1:04 PM
ashoat added inline comments.
web/modals/threads/create/compose-subchannel-modal.react.js
102–106 ↗(On Diff #31998)

Do we have anything equivalent on native? Or is it not possible to create an announcement channel on native yet?

This revision is now accepted and ready to land.Oct 16 2023, 1:54 PM
web/modals/threads/create/compose-subchannel-modal.react.js
102–106 ↗(On Diff #31998)

Not possible yet I believe, it's tracked in ENG-1887