Page MenuHomePhabricator
Differential D5405

[native] [12/34] ReactNav6: use NavHelpers for type parameter name instead of NavProp
ClosedPublic
Actions

Authored by ashoat on Oct 20 2022, 6:13 AM.
Tags
None
Referenced Files
F6143172: D5405.diff
Tue, Apr 22, 8:06 AM
Unknown Object (File)
Sat, Apr 19, 11:15 PM
Unknown Object (File)
Thu, Apr 17, 5:07 PM
Unknown Object (File)
Thu, Apr 17, 4:12 AM
Unknown Object (File)
Thu, Apr 17, 3:44 AM
Unknown Object (File)
Thu, Apr 17, 3:12 AM
Unknown Object (File)
Wed, Apr 16, 12:53 AM
Unknown Object (File)
Wed, Apr 9, 3:17 AM
View All Files
Subscribers
• abosh

Details

Reviewers
atul
tomek
bartek
Commits
rCOMM07a4b1ccb98a: [native] [12/34] ReactNav6: use NavHelpers for type parameter name instead of…
Summary

Following the changes in the last three diffs, we want to only use NavProp for actual navigation props (corresponding to a specific RouteName), and instead use the name NavHelpers for cases where we can't infer a specific RouteName.

Depends on D5404

Test Plan

Flow + ran the app on my phone and played around with it

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
native/
flow-typed/
npm/
@react-navigation/
32 lines
32 lines
32 lines
32 lines
32 lines

Diff 17683

View Options

native/flow-typed/npm/@react-navigation/bottom-tabs_v5.x.x.js

Loading...
View Options

native/flow-typed/npm/@react-navigation/devtools_v5.x.x.js

Loading...
View Options

native/flow-typed/npm/@react-navigation/material-top-tabs_v5.x.x.js

Loading...
View Options

native/flow-typed/npm/@react-navigation/native_v5.x.x.js

Loading...
View Options

native/flow-typed/npm/@react-navigation/stack_v5.x.x.js

Loading...