Page MenuHomePhabricator

[keyserver] Add Thread Invitation box to the frame
ClosedPublic

Authored by will on Nov 21 2024, 11:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jan 5, 8:18 AM
Unknown Object (File)
Sun, Jan 5, 5:50 AM
Unknown Object (File)
Thu, Jan 2, 4:52 AM
Unknown Object (File)
Wed, Dec 25, 3:21 PM
Unknown Object (File)
Wed, Dec 25, 3:21 PM
Unknown Object (File)
Wed, Dec 25, 3:21 PM
Unknown Object (File)
Wed, Dec 25, 3:21 PM
Unknown Object (File)
Wed, Dec 25, 3:21 PM
Subscribers

Details

Summary

This adds the thread invitation box to the frame image

Depends on D13985

Test Plan

Tested on the warpcast frame validator and confirmed that the thread invitation box appeared properly

image.png (926×1 px, 584 KB)

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

will edited the test plan for this revision. (Show Details)
will requested review of this revision.Nov 21 2024, 11:49 PM

Can you move the conditional outside the JSX before landing? Let me know if this feedback is confusing

keyserver/src/frog/frog.js
99–110 ↗(On Diff #45935)

We generally avoid this sort of conditional in our codebase

This revision is now accepted and ready to land.Nov 23 2024, 6:27 AM
keyserver/src/frog/frog.js
48–51 ↗(On Diff #45935)

What username is being displayed here, Farcaster or Comm?

If it's a Farcaster username, why do we need to do this? I thought you indicated the usernames were at most 16 characters?

keyserver/src/frog/frog.js
48–51 ↗(On Diff #45935)

In the linear comment below I specify that there's the possibility of people creating their own Comm frame url with an unusually long username as a parameter.

Unlikely but I included this just in case to prevent anyone from being able to render a frame with a username that goes out of the Thread Invitation box