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

#1633 fix allow compression Android #1634

Merged

Conversation

MateusHBR
Copy link
Contributor

The allowCompression param do not have any effect on Android

#1633

@MateusHBR MateusHBR changed the title #1633 fix allow compression #1633 fix allow compression Android Nov 22, 2024
Copy link
Owner

@miguelpruivo miguelpruivo left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@miguelpruivo
Copy link
Owner

Can you resolve the conflicts and also bump the patch version? Thanks.

@miguelpruivo miguelpruivo self-assigned this Dec 11, 2024
@MateusHBR
Copy link
Contributor Author

@miguelpruivo Done!

@loql-duc-luu
Copy link

there are new conflicts. @MateusHBR can you please resolve them? thanks

@MateusHBR
Copy link
Contributor Author

@loql-duc-luu and @miguelpruivo

Done again!

@loql-duc-luu
Copy link

thank you, @MateusHBR!

all green, @miguelpruivo. could you please approve and merge? thank you!

Copy link
Collaborator

@navaronbracke navaronbracke left a 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
Copy link
Collaborator

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)

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
Copy link
Collaborator

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.

@navaronbracke
Copy link
Collaborator

I went ahead and fixed the version + formatting issues. Once the checks are green this is LGTM.

@navaronbracke navaronbracke merged commit 79aaea6 into miguelpruivo:master Feb 11, 2025
3 checks passed
@MateusHBR MateusHBR deleted the fix-allow-compression-android branch February 11, 2025 12:53
@loql-duc-luu
Copy link

thanks so much, @navaronbracke 🙏

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.

4 participants