Page MenuHomePhabricator
Differential D5412

[native] [19/34] ReactNav6: Update UseNavigationBuilder type
ClosedPublic
Actions

Authored by ashoat on Oct 20 2022, 6:17 AM.
Tags
None
Referenced Files
F3249063: D5412.diff
Fri, Nov 15, 12:14 PM
Unknown Object (File)
Mon, Nov 11, 1:51 AM
Unknown Object (File)
Fri, Nov 8, 8:42 PM
Unknown Object (File)
Fri, Nov 8, 4:37 PM
Unknown Object (File)
Fri, Nov 8, 10:02 AM
Unknown Object (File)
Fri, Nov 8, 8:54 AM
Unknown Object (File)
Fri, Nov 8, 8:47 AM
Unknown Object (File)
Fri, Nov 8, 8:39 AM
View All Files
Subscribers
• abosh

Details

Reviewers
atul
tomek
bartek
Commits
rCOMMf5a7d0a78bf9: [native] [19/34] ReactNav6: Update UseNavigationBuilder type
Summary

We can unify this to use the new NavigatorProps type introduced in D5409.

We can see the TypeScript types upstream use the same construction.

Depends on D5411

Test Plan

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

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

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

Diff 17776

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...