-
-
Notifications
You must be signed in to change notification settings - Fork 173
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
File Copy Implementation #61
Conversation
*Total -- 68.92kb -> 55.78kb (19.06%) /Screenshot from 2020-07-12 13-30-51.png -- 42.47kb -> 30.83kb (27.4%) /app/src/main/res/mipmap-xxxhdpi/ic_launcher.png -- 10.40kb -> 9.54kb (8.33%) /app/src/main/res/mipmap-xxhdpi/ic_launcher.png -- 7.72kb -> 7.27kb (5.88%) /app/src/main/res/mipmap-hdpi/ic_launcher.png -- 3.51kb -> 3.43kb (2.14%) /app/src/main/res/mipmap-xhdpi/ic_launcher.png -- 4.81kb -> 4.71kb (2.07%) Signed-off-by: ImgBotApp <[email protected]>
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.
Thank you for contributing to this project. This is your first pull request and we are so glad to have you onboard. We will review the request and get back to you soon.
Can you please add the reason why did you add |
…wnload directory when targetSdkVersion > 30 (Android 11)
[ImgBot] Optimize images
Fix bug: Storage permissions not required to save file in Download
You need merge this PR. As it's handle write external storage permission above Android 11 @afreakyelf |
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.
Hi @rtsketo can you update your PR with latest changes please.
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days. |
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days. |
Code Refinements:
WRITE_EXTERNAL_STORAGE
, taking into account the different API levels and their permission models. This is critical for Android 6.0 (Marshmallow) and above, where runtime permissions are required.copyFile(..)
. This addresses a need to manipulate files within the app's scope effectively, following best practices for file storage.downloadFile(..)
function has been improved to use Environment.getExternalStoragePublicDirectory(), which is a more standardized way to access shared storage in a manner consistent with Android's scoped storage model.Code Cleanup:
Gradle Updates:
Manifest Adjustments:
Visual Assets: