Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Fix hiding header on IOS not working #339

Merged
merged 1 commit into from
Jan 9, 2020

Conversation

Phillip-Cognativ
Copy link
Contributor

Overview

Small fix for broken prop: #301 (comment)

Test Plan

Copy link
Owner

@mmazzarolo mmazzarolo left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@mmazzarolo mmazzarolo merged commit 3f4f7ba into mmazzarolo:next-major Jan 9, 2020
@mmazzarolo
Copy link
Owner

@Phillip-Cognativ thank you! available in v8.0.6!

mmazzarolo added a commit that referenced this pull request Jan 16, 2020
* feat: Moved to @react-native-community pickers

BREAKING CHANGE

* Bump version

BREAKING CHANGE: bumping to 8.0.0

* docs: Added v8 info

* docs: Update README.md

* docs: Update README.md

* docs: Update README.md

* Update README.md

* docs: Update README.md

* fix: Fixed modal styling (#306)

* fix: Updated TypeScript definitions for v8.0.0 (#307)

* Updated dependencies

* Fix: Wrong version bump

* v8.0.1

* fix: Use getDate() instead of getDay()

* v8.0.2

* Type custom components in props as React.ComponentType / PropTypes.elementType (#314)

* v8.0.3

* fix: Rename onModalHide to onHide Modal prop to make onHide callback work on iOS (#316)

* Export TS interface

* v8.0.4

* Mentioned Expo instructions in the README.md

* feat: Added custom component types (#336)

* v8.0.5

* fix: Fix hiding header on IOS not working (#339)

* v8.0.6

* Set the library version to 8.x.x for Expo setup (#342)

It would probably makes things easier for people migrating from version 7.

Co-authored-by: Michiel De Mey <[email protected]>
Co-authored-by: Christiaan van Bemmel <[email protected]>
Co-authored-by: Alexey Bulavka <[email protected]>
Co-authored-by: Simonas Gildutis <[email protected]>
Co-authored-by: Phillip-Cognativ <[email protected]>
Co-authored-by: Victor Goya <[email protected]>
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