Page MenuHomePhabricator
Differential D5422

[native] [29/34] ReactNav6: Merge tabBarOptions into options
ClosedPublic
Actions

Authored by ashoat on Oct 20 2022, 6:19 AM.
Tags
None
Referenced Files
F3759131: D5422.diff
Fri, Jan 10, 10:05 AM
F3754546: D5422.id17700.diff
Fri, Jan 10, 3:24 AM
F3754520: D5422.id17700.diff
Fri, Jan 10, 3:19 AM
Unknown Object (File)
Fri, Dec 27, 5:19 PM
Unknown Object (File)
Fri, Dec 27, 5:19 PM
Unknown Object (File)
Fri, Dec 27, 5:17 PM
Unknown Object (File)
Fri, Dec 27, 4:57 PM
Unknown Object (File)
Thu, Dec 12, 10:00 PM
View All Files
Subscribers
• abosh

Details

Reviewers
atul
tomek
bartek
Commits
rCOMM3924e11a9557: [native] [29/34] ReactNav6: Merge tabBarOptions into options
Summary

Context on change here.

Depends on D5421

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

Event Timeline

ashoat created this revision.Oct 20 2022, 6:19 AM
Herald added a subscriber: • abosh. · View Herald TranscriptOct 20 2022, 6:19 AM
ashoat added a child revision: D5423: [native] [30/34] ReactNav6: Remove tabBarVisible option.Oct 20 2022, 6:20 AM
Harbormaster returned this revision to the author for changes because remote builds failed.Oct 20 2022, 6:42 AM
Harbormaster failed remote builds in B12909: Diff 17700!
ashoat requested review of this revision.Oct 20 2022, 7:25 AM

Build failures are caused by use of @react-navigation/elements in D5420, which isn't introduced until D5427. Sorry!!

atul accepted this revision.Oct 20 2022, 12:55 PM
This revision is now accepted and ready to land.Oct 20 2022, 12:55 PM
This revision was landed with ongoing or failed builds.Oct 20 2022, 1:47 PM
Closed by commit rCOMM3924e11a9557: [native] [29/34] ReactNav6: Merge tabBarOptions into options (authored by ashoat). · Explain Why
This revision was automatically updated to reflect the committed changes.
ashoat added a commit: rCOMM3924e11a9557: [native] [29/34] ReactNav6: Merge tabBarOptions into options.

Revision Contents
Changeset List

PathSize
native/
flow-typed/
npm/
@react-navigation/
37 lines
37 lines
37 lines
37 lines
37 lines
37 lines
navigation/
13 lines

Diff 17785

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/elements_v1.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...
View Options

native/navigation/app-navigator.react.js

Loading...