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

feat: Add "any" mediaType support for openCamera on iOS #1754

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gabrieldonadel
Copy link

@gabrieldonadel gabrieldonadel commented Mar 21, 2022

Description

This adds support for the any mediaType when using the openCamera function on iOS, allowing users to either take a picture or record a video. Unfortunately with the current approach, there's no way to add support for the any mediaType on Android, as we use different intents depending on the selected media type and android does not have an intent that supports both taking images and recording videos.

@gabrieldonadel gabrieldonadel changed the title feat: Add "any" mediaType support for openCamera feat: Add "any" mediaType support for openCamera on iOS Mar 21, 2022
Copy link

@felippepuhle felippepuhle left a comment

Choose a reason for hiding this comment

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

Looks great!

@gabrieldonadel
Copy link
Author

@ivpusic could you please review this?

@thanggggg1
Copy link

@gabrieldonadel how can i import your lib

@gabrieldonadel
Copy link
Author

@gabrieldonadel how can i import your lib

You can patch react-native-image-crop-picker and apply these changes manually using patch-package

@brianMxBm
Copy link

@gabrieldonadel I'm assuming patch-package is the only way to get this change in considering this didn't make to an official release?

@gabrieldonadel
Copy link
Author

@gabrieldonadel I'm assuming patch-package is the only way to get this change in considering this didn't make to an official release?

Yeah @mxbm, unfortunately @ivpusic has not reviewed this yet

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.

None yet

4 participants