-
Notifications
You must be signed in to change notification settings - Fork 716
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
#1633 fix allow compression Android #1634
#1633 fix allow compression Android #1634
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks.
Can you resolve the conflicts and also bump the patch version? Thanks. |
@miguelpruivo Done! |
there are new conflicts. @MateusHBR can you please resolve them? thanks |
@loql-duc-luu and @miguelpruivo Done again! |
thank you, @MateusHBR! all green, @miguelpruivo. could you please approve and merge? thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some minor feedback about the version for this fix, and some formatting issues.
CHANGELOG.md
Outdated
@@ -1,3 +1,7 @@ | |||
## 8.3.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This changelog entry will need to be updated to the next hotfix version. (you will need to move it to the top of this file and change the version)
android/src/main/java/com/mr/flutter/plugin/filepicker/FilePickerDelegate.java
Outdated
Show resolved
Hide resolved
android/src/main/java/com/mr/flutter/plugin/filepicker/FilePickerPlugin.java
Outdated
Show resolved
Hide resolved
pubspec.yaml
Outdated
@@ -3,7 +3,7 @@ description: A package that allows you to use a native file explorer to pick sin | |||
homepage: https://github.com/miguelpruivo/plugins_flutter_file_picker | |||
repository: https://github.com/miguelpruivo/flutter_file_picker | |||
issue_tracker: https://github.com/miguelpruivo/flutter_file_picker/issues | |||
version: 8.3.1 | |||
version: 8.3.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This version will need to be updated to the next hotfix version.
I went ahead and fixed the version + formatting issues. Once the checks are green this is LGTM. |
thanks so much, @navaronbracke 🙏 |
The allowCompression param do not have any effect on Android
#1633