Page MenuHomePhabricator
Differential D11682

[native] introduce screen options to tag farcaster channel stack navigator
ClosedPublic
Actions

Authored by ginsu on Wed, Apr 17, 11:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 27, 2:34 AM
Unknown Object (File)
Wed, Apr 24, 8:58 PM
Unknown Object (File)
Wed, Apr 24, 3:09 PM
Unknown Object (File)
Wed, Apr 24, 12:26 PM
Unknown Object (File)
Mon, Apr 22, 9:29 AM
Unknown Object (File)
Thu, Apr 18, 6:25 PM
Unknown Object (File)
Thu, Apr 18, 6:25 PM
Unknown Object (File)
Thu, Apr 18, 6:24 PM
View All 10 Files
Subscribers
ashoat
tomek

Details

Reviewers
atul
inka
ashoat
Commits
rCOMM409e47a1b60b: [native] introduce screen options to tag farcaster channel stack navigator
Summary

This diff adds the screen options to the TagFarcasterChannelNavigator and the TagFarcasterChannel screen to match the UI/UX in the other community action screens

Linear task: https://linear.app/comm/issue/ENG-7540/introduce-navigational-logic-for-tagfarcasterchannelmodal

Depends on D11662

Test Plan

Please see the demo video below (the other UI elements in the TagFarcasterChannel will be introduced in the next diff)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ginsu created this revision.Wed, Apr 17, 11:04 PM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptWed, Apr 17, 11:04 PM
ginsu edited the summary of this revision. (Show Details)Wed, Apr 17, 11:10 PM
ginsu edited the test plan for this revision. (Show Details)
ginsu added reviewers: atul, inka.
ginsu added a parent revision: D11662: [native] introduce barebones TagFarcasterChannelNavigator + TagFarcasterChannel screen.
ginsu attached a referenced file: F1567055: Screen Recording 2024-04-18 at 2.07.54 AM.mov. (Show Details)
ginsu added a reviewer: ashoat.Wed, Apr 17, 11:16 PM

Making @ashoat a blocking reviewer since this diff introduces copy changes

native/community-settings/tag-farcaster-channel/tag-farcaster-channel-navigator.react.js
33 ↗(On Diff #39206)

This copy needs to be reviewed

Harbormaster completed remote builds in B28282: Diff 39206.Wed, Apr 17, 11:21 PM
ginsu requested review of this revision.Wed, Apr 17, 11:21 PM
ginsu added a child revision: D11683: [native] implement UI for selecting a farcaster channel.Wed, Apr 17, 11:23 PM
ashoat accepted this revision.Thu, Apr 18, 11:31 AM
This revision is now accepted and ready to land.Thu, Apr 18, 11:31 AM
Closed by commit rCOMM409e47a1b60b: [native] introduce screen options to tag farcaster channel stack navigator (authored by ginsu). · Explain WhyThu, Apr 18, 12:06 PM
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMM409e47a1b60b: [native] introduce screen options to tag farcaster channel stack navigator.

Revision Contents
Changeset List

PathSize
native/
community-settings/
tag-farcaster-channel/
27 lines

Diff 39237

View Options

native/community-settings/tag-farcaster-channel/tag-farcaster-channel-navigator.react.js

Loading...