Skip to content

Fix: Add legacy storage permission check for saving covers#3106

Open
leodyversemilla07 wants to merge 1 commit intomihonapp:mainfrom
leodyversemilla07:fix/save-cover-storage-permission
Open

Fix: Add legacy storage permission check for saving covers#3106
leodyversemilla07 wants to merge 1 commit intomihonapp:mainfrom
leodyversemilla07:fix/save-cover-storage-permission

Conversation

@leodyversemilla07
Copy link
Copy Markdown
Contributor

@leodyversemilla07 leodyversemilla07 commented Mar 21, 2026

Summary

Fixes #3097 - Saving manga covers and reader images would fail silently without proper storage permission on Android versions below Android 10 (Q).

Changes

  • Added rememberLegacyStoragePermissionState composable in Permissions.kt to track WRITE_EXTERNAL_STORAGE permission state
  • Added permission check and request dialog in MangaScreen when saving manga covers
  • Added permission check and request dialog in ReaderActivity when saving reader images
  • Automatically grants permission on Android Q+ since those versions use scoped storage

Testing

  • Tested on Android 9 and below - permission dialog appears before saving
  • Tested on Android 10+ - save works without permission dialog (scoped storage)
  • Verified both manga cover saving and reader image saving flows
  • Tested all base themes and tablet mode

@leodyversemilla07 leodyversemilla07 force-pushed the fix/save-cover-storage-permission branch 6 times, most recently from a6c19ef to 6d33e78 Compare March 21, 2026 02:23
Request WRITE_EXTERNAL_STORAGE permission on Android < Q when saving manga covers or reader images. Added rememberLegacyStoragePermissionState composable to track permission state and show dialog when needed.
@leodyversemilla07 leodyversemilla07 force-pushed the fix/save-cover-storage-permission branch from 6d33e78 to 7e7c5bd Compare March 26, 2026 08:49
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.

App does not ask for storage perms

1 participant