Skip to content

fix(upload): limited file access - #17567

Open
alperozturk96 wants to merge 1 commit into
masterfrom
fix/limited-file-access-file-selection
Open

fix(upload): limited file access#17567
alperozturk96 wants to merge 1 commit into
masterfrom
fix/limited-file-access-file-selection

Conversation

@alperozturk96

Copy link
Copy Markdown
Collaborator

Steps to reproduce

  1. Clean install of the app
  2. Storage permissions popup select "Media read-only"
  3. Select "Allow limited access" for some specific files
  4. Tap FAB button that opens local file listing
  5. Select same media previously allowed and observe while select / unselect / select all media file
  6. Tap upload button

Actual Result:

  1. When selecting/deselecting media files or using the Select All option, the UI appears to hang. In some cases, media files cannot be manually selected or deselected.
  2. Clicking the Upload button is unresponsive, and the upload process does not start. Only the Cancel button works.
  3. The Upload option is also not clickable when accessed from the footer section.

Expected Result:
With Read-Only and Limited Access permissions, users should be able to upload files without any issues.

Changes

  • Fixes UI lag
  • If user selects limited file access do not shows storage permission banner

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown

blue-Light-Screenshot test failed, but no output was generated. Maybe a preliminary stage failed.

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown

Codacy

SpotBugs

CategoryBaseNew
Bad practice3535
Correctness5858
Dodgy code195195
Experimental11
Malicious code vulnerability22
Multithreaded correctness2626
Performance3737
Security1919
Total373373

@daniele-verducci daniele-verducci left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey Alper, I couldn't reproduce the issue on the master branch, but I tested this one and seems to work properly. I tested on Android 16 granting different levels of permissions and on Android 10, that doesn't support for granular permissions, and it worked as I would expect.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants