Page MenuHomePhabricator

[native] Update @react-navigation/drawer libdef for React Nav 6
ClosedPublic

Authored by ashoat on Nov 4 2022, 10:20 AM.
Tags
None
Referenced Files
F3523950: D5538.diff
Mon, Dec 23, 10:45 AM
Unknown Object (File)
Mon, Dec 16, 1:46 AM
Unknown Object (File)
Mon, Dec 9, 9:37 PM
Unknown Object (File)
Thu, Nov 28, 6:11 AM
Unknown Object (File)
Sun, Nov 24, 11:49 AM
Unknown Object (File)
Sun, Nov 24, 11:49 AM
Unknown Object (File)
Nov 22 2024, 8:33 AM
Unknown Object (File)
Nov 22 2024, 8:33 AM
Subscribers

Details

Summary

See D5537 and ENG-2175 for context. Will describe changes and where they came from inline.

Depends on D5537

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Branch
ashoat/react-navigation-drawer
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Nov 4 2022, 10:29 AM
Harbormaster failed remote builds in B13184: Diff 18090!

I referenced the TypeScript types when writing this diff.

You only need to review the drawer libdef, as the others are copy-pasted from there.

native/flow-typed/npm/@react-navigation/drawer_v6.x.x.js
745
1974–2004
  1. Some of these are new options migrated from drawerContentOptions in React Nav 6
  2. Some of these got their name changed at some point prior to React Nav 6. I found the new names via API docs and TypeScript types
1994–1995
2005

New property here in React Nav 6, details here

2045–2058
2069

React Nav 6 removes this

2069

TypeScript types only have these three props. Upgrade notes only mention removing one prop, so the others were probably removed earlier and the libdef is probably just out of date

2071

New property here in React Nav 6, details here

2073–2086

Most of these props were moved to options at some point – you can see them in the TypeScript types. React Nav 6 upgrade docs don't mention this, so presumably the change occurred earlier

2082
2297
Harbormaster returned this revision to the author for changes because remote builds failed.Nov 4 2022, 10:42 AM
Harbormaster failed remote builds in B13185: Diff 18091!
ashoat requested review of this revision.Nov 5 2022, 6:10 AM

Android CI is broken right now, so bypassing it

Thanks for annotating the relevant changes

This revision is now accepted and ready to land.Nov 7 2022, 7:02 AM