Skip to content
This repository was archived by the owner on Aug 5, 2025. It is now read-only.

Add additional (optional) navigation tab(s)#184

Merged
cmjarreau merged 11 commits intomasterfrom
add-optional-parameter-to-rootstack-2
May 9, 2023
Merged

Add additional (optional) navigation tab(s)#184
cmjarreau merged 11 commits intomasterfrom
add-optional-parameter-to-rootstack-2

Conversation

@cmjarreau
Copy link
Contributor

Changes

  • add additionalNavTab param to DeveloperConfigProvider and map in TabNavigator

Screenshots

Screen Shot 2023-05-08 at 2 29 32 PM

@coveralls
Copy link

coveralls commented May 8, 2023

Pull Request Test Coverage Report for Build 4927071540

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 12 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.09%) to 79.703%

Files with Coverage Reduction New Missed Lines %
src/components/tiles/TilesList.tsx 12 61.64%
Totals Coverage Status
Change from base Build 4897230712: -0.09%
Covered Lines: 1332
Relevant Lines: 1650

💛 - Coveralls

NotificationsTab: undefined;
SettingsTab: NavigatorScreenParams<SettingsStackParamList>;
} & {
[keyof: string]: undefined;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm wondering if there is another way to do this where we could preserve these types without adding the equivalent of an any here. I'm not totally sure what that would be so I am approving as is.

bruce-glazier
bruce-glazier previously approved these changes May 9, 2023

export const HelloWorldScreen = () => {
return (
<Text style={{ paddingTop: '50%', paddingLeft: '40%' }}>Hello, world!</Text>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: Can you fix this warning?

@cmjarreau cmjarreau merged commit 6601b1d into master May 9, 2023
@cmjarreau cmjarreau deleted the add-optional-parameter-to-rootstack-2 branch May 9, 2023 14:48
@github-actions
Copy link

github-actions bot commented May 9, 2023

🎉 This PR is included in version 1.18.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions bot added the released Has been released to the package repository (NPM, etc) label May 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

released Has been released to the package repository (NPM, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants