Paths

Table of Contentst

Differential D13813

[keyserver] remove commbot after creating community
Needs RevisionPublic
Actions

Authored by will on Tue, Oct 29, 4:29 PM.

Tags

None

Referenced Files

None

Subscribers

Details

Reviewers

varun
ashoat

Summary

This removes commbot after it creates the community. There will always be another keysrver admin as it will either be the tagger, channel lead, or the keyserver admin.

Depends on D13775

Test Plan

Tested by auto-creating a community for the dgn channel and confirming that after commbot had created the community, the tagger userID was promoted to admin and commbot did not appear on members.

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

will created this revision.Tue, Oct 29, 4:29 PM

Herald added a subscriber: tomek. · View Herald TranscriptTue, Oct 29, 4:29 PM

rebase

will edited the summary of this revision. (Show Details)Tue, Oct 29, 4:33 PM

will added a parent revision: D13775: [keyserver] add community creation logic to farcaster webhook responder.

will added a child revision: D13812: [keyserver] Add dontCheckJoinPermissions override option to createThread for sidebar.Tue, Oct 29, 4:36 PM

Harbormaster completed remote builds in B32416: Diff 45443.Tue, Oct 29, 4:47 PM

Harbormaster completed remote builds in B32417: Diff 45444.Tue, Oct 29, 4:50 PM

will requested review of this revision.Tue, Oct 29, 4:50 PM

See feedback in D13812 – we wouldn't need to remove commbot if we never add them to the thread

This revision now requires changes to proceed.Tue, Oct 29, 7:08 PM

will removed a child revision: D13812: [keyserver] Add dontCheckJoinPermissions override option to createThread for sidebar.Wed, Oct 30, 12:16 AM

Revision Contents
Changeset List

Path

Size

keyserver/

src/

responders/

farcaster-webhook-responders.js

7 lines

Diff 45444

keyserver/src/responders/farcaster-webhook-responders.js

Loading...