Page MenuHomePhabricator

[native] community joiner modal
Needs ReviewPublic

Authored by varun on Thu, Nov 21, 7:58 PM.
Tags
None
Referenced Files
F3346600: D13996.diff
Fri, Nov 22, 9:16 AM
F3339967: D13996.id45933.diff
Thu, Nov 21, 9:05 PM
F3339960: D13996.id.diff
Thu, Nov 21, 9:05 PM
F3339958: D13996.diff
Thu, Nov 21, 9:05 PM
Subscribers

Details

Reviewers
ashoat
Summary

Depends on D13995

we'll navigate to this modal from the community drawer button in D13987 and from the bottom sheet in D13986

Test Plan

tested that the tabs work as expected and all the community threads display in the right tab. tested the join/leave behavior in previous diff, which also has a video with the modal

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

native/components/community-joiner-modal.react.js
134

this is obviously wrong, but was having some trouble adding a libdef for react-native-tab-view. i couldn't figure out how to import the Animated type from 'react-native' in my libdef

figured I'd put this diff up before spending too much time on this, since @ashoat might have some ideas here

134

by wrong, I mean not how we usually resolve flow issues

native/components/community-joiner-modal.react.js
19

we don't want to display an empty modal so we make sure we have the list of communities first before we navigate to this modal

30
varun requested review of this revision.Thu, Nov 21, 8:18 PM

btw, this diff replaces D13493, which I plan on abandoning. i think i've applied all the feedback from there, but referencing here just in case