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: assets is actually optional #1772

Merged
merged 1 commit into from
Jul 15, 2021

Conversation

msegers
Copy link
Contributor

@msegers msegers commented Jul 15, 2021

Typings are incorrect, and will cause potentially crashing apps if code is implemented incorrectly :-).

Thanks for submitting a PR! Please read these instructions carefully:

  • Explain the motivation for making this change.
  • Provide a test plan demonstrating that the code is solid.
  • Match the code formatting of the rest of the codebase.
  • Target the master branch, NOT a "stable" branch.

Motivation (required)

The typings are incorrect, caused a bug in our production app.

What existing problem does the pull request solve?
See Motivation

Test Plan (required)

Implement gallery picker, cancel picking see no assets if you log or try to use the field

A good test plan has the exact commands you ran and their output, provides screenshots or videos if the pull request changes UI or updates the website. See What is a Test Plan? to learn more.

If you have added code that should be tested, add tests.

Typings are incorrect, and will cause potentially crashing apps if code is implemented incorrectly :-).
@msegers
Copy link
Contributor Author

msegers commented Jul 15, 2021

I do not see a master branch, I guess main is ok?

@Johan-dutoit Johan-dutoit merged commit 6169753 into react-native-image-picker:main Jul 15, 2021
Johan-dutoit pushed a commit that referenced this pull request Jul 15, 2021
## [4.0.6](v4.0.5...v4.0.6) (2021-07-15)

### Bug Fixes

* mark assets as optional ([#1772](#1772)) ([6169753](6169753))
@Johan-dutoit
Copy link
Collaborator

🎉 This PR is included in version 4.0.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

MattDennyir pushed a commit to MattDennyir/react-native-image-picker that referenced this pull request Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants