Page MenuHomePhabricator

[web] introduce TagFarcasterChannelModal
ClosedPublic

Authored by ginsu on Jun 20 2024, 8:16 PM.
Tags
None
Referenced Files
F3369948: D12530.diff
Tue, Nov 26, 12:18 AM
Unknown Object (File)
Fri, Nov 22, 10:31 PM
Unknown Object (File)
Wed, Nov 13, 3:28 AM
Unknown Object (File)
Wed, Nov 13, 3:28 AM
Unknown Object (File)
Sat, Nov 9, 2:19 PM
Unknown Object (File)
Oct 22 2024, 9:50 PM
Unknown Object (File)
Oct 12 2024, 1:52 PM
Unknown Object (File)
Oct 12 2024, 1:52 PM
Subscribers

Details

Summary

This diff introduces the TagFarcasterChannelModal component. This modal will show the selected farcaster channel if there is one tag otherwise it will let the user create a tag. Subsequent diffs in this stack will extend this modal

Test Plan

Please see the screenshot below

w/ tagged channel

Screenshot 2024-06-20 at 11.18.48 PM.png (2×3 px, 801 KB)

w/o tagged channel

Screenshot 2024-06-20 at 11.17.36 PM.png (2×3 px, 895 KB)

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu edited the test plan for this revision. (Show Details)
ginsu added reviewers: kamil, inka.
ginsu edited the summary of this revision. (Show Details)
ginsu added a reviewer: ashoat.

copy review

web/tag-farcaster-channel/tag-farcaster-channel-modal.react.js
43 ↗(On Diff #41588)
ginsu edited the summary of this revision. (Show Details)

update

web/tag-farcaster-channel/tag-farcaster-channel-modal.react.js
28 ↗(On Diff #41589)
41 ↗(On Diff #41589)
43 ↗(On Diff #41589)
45 ↗(On Diff #41589)
ginsu requested review of this revision.Jun 20 2024, 8:42 PM

Looks good, but can you dedup the copy-pasted strings before landing?

This revision is now accepted and ready to land.Jun 27 2024, 9:51 AM
This revision was automatically updated to reflect the committed changes.