Page MenuHomePhabricator

[native] button in community drawer for exploring communities
ClosedPublic

Authored by varun on Nov 21 2024, 12:15 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 27, 10:49 AM
Unknown Object (File)
Thu, Mar 27, 5:04 AM
Unknown Object (File)
Thu, Mar 27, 3:53 AM
Unknown Object (File)
Wed, Mar 26, 3:37 AM
Unknown Object (File)
Sun, Mar 23, 12:59 PM
Unknown Object (File)
Sat, Mar 22, 4:24 PM
Unknown Object (File)
Feb 26 2025, 9:23 AM
Unknown Object (File)
Feb 22 2025, 8:28 PM
Subscribers

Details

Summary

https://linear.app/comm/issue/ENG-9850/add-new-button-in-profile-section-on-native-that-navigates-to

We need this button so that users can navigate back to the directory after it's been dismissed

Depends on D13986

Test Plan

button is pressable, but currently does nothing. in a subsequent diff we will navigate to the community directory from here onPress

Simulator Screenshot - iPhone 16 Pro Max - 2024-11-21 at 03.13.14.png (2×1 px, 163 KB)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

native/navigation/community-drawer-content.react.js
173 ↗(On Diff #45918)

we can remove this once the button does something

ashoat added inline comments.
native/navigation/community-drawer-content.react.js
173 ↗(On Diff #45918)

Seems like you're going to be gating your handler component (that shows the modal) somehow... would it make sense to have a gate variable, so both checks are associated and can be flipped at once?

188–189 ↗(On Diff #45918)

Not sure these two lines are necessary

This revision is now accepted and ready to land.Nov 21 2024, 4:04 PM
native/navigation/root-navigator.react.js
85

This is a bad place to put this, and we should never have export const variables in most files, especially a file like this one