Page MenuHomePhabricator

[native] move BottomSheetModalProvider to inside NavigationContainer
ClosedPublic

Authored by ginsu on Sep 6 2023, 1:01 PM.
Tags
None
Referenced Files
F3133542: D9094.id31003.diff
Sat, Nov 2, 12:26 PM
Unknown Object (File)
Fri, Nov 1, 12:47 PM
Unknown Object (File)
Wed, Oct 30, 5:17 AM
Unknown Object (File)
Sun, Oct 20, 9:30 AM
Unknown Object (File)
Sun, Oct 20, 9:30 AM
Unknown Object (File)
Sun, Oct 20, 9:30 AM
Unknown Object (File)
Sun, Oct 20, 9:29 AM
Unknown Object (File)
Sun, Oct 20, 9:24 AM
Subscribers

Details

Summary

In D8935 I add the BottomSheetModalProvider to the root component; however, I made a mistake where I added it in the wrong place. Since we are navigating to our bottomsheets we actually want this provider to be inside NavigationContainer

Depends on D9101

Test Plan

Was initially getting the following error; however, after I created this diff the error does not appear and I was able to navigate to the

Screenshot 2023-09-06 at 3.48.35 PM.png (1×1 px, 803 KB)

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu requested review of this revision.Sep 6 2023, 1:20 PM
ginsu edited the test plan for this revision. (Show Details)
ginsu added reviewers: atul, inka.

Sequenced this diff after D9101 in the stack so that this diff felt less "random" and that there should be more context for this after reading D9101

This revision is now accepted and ready to land.Sep 8 2023, 3:52 AM
This revision was landed with ongoing or failed builds.Sep 12 2023, 2:31 PM
This revision was automatically updated to reflect the committed changes.