-
Notifications
You must be signed in to change notification settings - Fork 341
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
New feature multiple selection was added. #291
base: master
Are you sure you want to change the base?
Conversation
New feature to select multiple files with the option multiplePicker, async task was changed by coroutine worker, this is the new reccommended way by google to make async task. Some code refactor. Commented // mCropProvider.delete() because, it was removing image after generate, it was causing an error that image wasn't available when client wanted to consumed.
It's working correctly! Once the property is lost so that it returns several images, the callback is called as many times as images have been selected, and they are added to the files/DCIM folder |
Why is not merged ? It take too much time... |
This should be merged. Thanks |
I would love this feature to be merged! |
Be great to merge this! |
I would be really love to have this option. Could you tell me when we have it? |
@EkaterinaT89 Probably never since the last merged in this repository is 2021 |
Why was this not merged? |
🚀 Description
New feature was added, multiple picker selection.
This feature is only availabe as picker and compression, not crop. The feature allows user to select multiple files from gallery and it's will be crompress, this feature is only for the new picker mode not legacy intent.
📄 Motivation and Context
The motivation is clear, give a feature neccesary multiple files selector.
🧪 How Has This Been Tested?
Tested with the example, but I haven't the neccesary know how to test properly.
📦 Types of changes
✅ Checklist