Skip to content

fix(types): add children to HeaderNavigationProps#5180

Merged
chasestarr merged 3 commits intouber:masterfrom
alexgorbatchev:fix-headernavigationprops
Oct 4, 2022
Merged

fix(types): add children to HeaderNavigationProps#5180
chasestarr merged 3 commits intouber:masterfrom
alexgorbatchev:fix-headernavigationprops

Conversation

@alexgorbatchev
Copy link
Contributor

@alexgorbatchev alexgorbatchev commented Sep 30, 2022

Description

HeaderNavigation.children is currently missing when paired with @types/react@18. This PR fixes addresses that.

Scope

Patch: Bug Fix

@codesandbox-ci
Copy link

codesandbox-ci bot commented Sep 30, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 2d09edc:

Sandbox Source
Basic usage Configuration

@alexgorbatchev alexgorbatchev force-pushed the fix-headernavigationprops branch from 724f7e3 to 9e8f079 Compare September 30, 2022 22:48
@alexgorbatchev alexgorbatchev changed the title fix(types): use React.PropsWithChilder with HeaderNavigationProps fix(types): use React.PropsWithChildren with HeaderNavigationProps Sep 30, 2022
@alexgorbatchev alexgorbatchev force-pushed the fix-headernavigationprops branch from 9e8f079 to f8c736c Compare October 4, 2022 16:15
@chasestarr chasestarr merged commit da4e460 into uber:master Oct 4, 2022
@alexgorbatchev alexgorbatchev changed the title fix(types): use React.PropsWithChildren with HeaderNavigationProps fix(types): add children to HeaderNavigationProps Oct 4, 2022
@alexgorbatchev alexgorbatchev deleted the fix-headernavigationprops branch October 6, 2022 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants