Page MenuHomePhabricator

[keyserver] create invite link based on newly created sidebar
AcceptedPublic

Authored by will on Wed, Oct 23, 10:42 PM.
Tags
None
Referenced Files
F3059498: D13783.id.diff
Thu, Oct 24, 3:42 AM
F3059240: D13783.id.diff
Thu, Oct 24, 3:28 AM
F3058989: D13783.diff
Thu, Oct 24, 2:50 AM
F3058988: D13783.diff
Thu, Oct 24, 2:49 AM
F3058046: D13783.id45365.diff
Wed, Oct 23, 10:57 PM
F3058043: D13783.id.diff
Wed, Oct 23, 10:57 PM
F3058038: D13783.diff
Wed, Oct 23, 10:57 PM
Subscribers

Details

Reviewers
varun
ashoat
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

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 requested review of this revision.Wed, Oct 23, 11:45 PM
keyserver/src/responders/farcaster-webhook-responders.js
255

Placeholder to be used in replying to cast

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