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

[BUG] Storage permission issue #22

Open
cohesive-flight opened this issue Apr 23, 2024 · 1 comment
Open

[BUG] Storage permission issue #22

cohesive-flight opened this issue Apr 23, 2024 · 1 comment
Labels
bug Something isn't working fritter backlog This issue was originally created in fritter help wanted Extra attention is needed P2 This is minor bug or a feature request that would benefit many users

Comments

@cohesive-flight
Copy link

Describe the bug
Quacker states it needs the storage permission when trying to set up a save directory, but no such permission is ever shown or requested.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'Download path'
  2. Tap on 'Choose'
  3. See the error message, "Permission not granted. Please try again after granting!"
  4. Go to the app's permissions
  5. See that no storage permission is displayed

Expected behavior
For Quacker to request the storage permission the moment the "Choose" button is tapped, like Squawker does.

Screenshots
Screenshot_20240423-045200
Screenshot_20240423-045150

Smartphone (please complete the following information):

  • Device: Pixel 7
  • OS: GrapheneOS (Android 14)
  • Version: v3.4.0

Additional info
This is nearly identical to jonjomckay/fritter#560, but in this case, I'm unable to grant it through App Manager using ADB, as not even the outdated read/write permissions are displayed on the 'Uses Permissions' tab, as shown in the second screenshot.

I'd like to suggest implementing scoped storage. This would avoid having to rely on outdated and unnecessarily invasive read/write permissions, as the app only needs access to a single directory.

@TheHCJ TheHCJ added bug Something isn't working P2 This is minor bug or a feature request that would benefit many users labels Apr 24, 2024
@TheHCJ
Copy link
Member

TheHCJ commented Apr 24, 2024

Thanks for reporting this bug, I'll look into it as soon as possible

@TheHCJ TheHCJ added help wanted Extra attention is needed fritter backlog This issue was originally created in fritter labels May 2, 2024
@TheHCJ TheHCJ closed this as completed May 16, 2024
@TheHCJ TheHCJ reopened this Jun 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fritter backlog This issue was originally created in fritter help wanted Extra attention is needed P2 This is minor bug or a feature request that would benefit many users
Projects
None yet
Development

No branches or pull requests

2 participants