[keyserver] create invite link based on newly created sidebar
ClosedPublic
Actions

Authored by will on Wed, Oct 23, 10:42 PM.

Details

Reviewers

varun
ashoat

Commits

rCOMMdb09962901fb: [keyserver] create invite link based on newly created sidebar

Summary

This creates an invite link for the newly created sidebar.

Depends on D13782

Test Plan

console logged the invitelink. On my iOS simulator dev build, plugged in the invite link to safari with https://comm.app/invite/<value> and confirmed I successfully joined the community and sidebar with a user originally not in either.

Diff Detail

Repository

rCOMM Comm

Branch

farcaster_bot

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

will created this revision.Wed, Oct 23, 10:42 PM

Herald added a subscriber: tomek. · View Herald TranscriptWed, Oct 23, 10:42 PM

will edited the summary of this revision. (Show Details)Wed, Oct 23, 10:42 PM

will added a parent revision: D13782: [keyserver] create message and sidebar based on cast.

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

Harbormaster returned this revision to the author for changes because remote builds failed.Wed, Oct 23, 10:46 PM

Harbormaster failed remote builds in B32360: Diff 45365!

will added a child revision: D13784: [lib] add post cast to neynar client.Wed, Oct 23, 11:43 PM

will requested review of this revision.Wed, Oct 23, 11:45 PM

Harbormaster completed remote builds in B32360: Diff 45365.Wed, Oct 23, 11:47 PM

will added inline comments.Thu, Oct 24, 6:18 AM

keyserver/src/responders/farcaster-webhook-responders.js
255	Placeholder to be used in replying to cast

ashoat accepted this revision.Thu, Oct 24, 6:22 AM

ashoat added inline comments.

keyserver/src/responders/farcaster-webhook-responders.js
247	Seems like you call this 3 times in this file. Why not lift it to the top-level scope so it can be called just once?

This revision is now accepted and ready to land.Thu, Oct 24, 6:22 AM

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

will removed a parent revision: D13812: [keyserver] Add dontCheckJoinPermissions override option to createThread for sidebar.

will added inline comments.Wed, Oct 30, 1:47 PM

keyserver/src/responders/farcaster-webhook-responders.js
247	Included in a rebase of https://phab.comm.dev/D13775

ashoat added inline comments.Wed, Oct 30, 2:53 PM

keyserver/src/responders/farcaster-webhook-responders.js
247	This diff depends on that one, though... don't you need to update this diff too? I would expect to see an updated version of this diff that doesn't introduce a new `commbotViewer`