-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Downloads prompts do not show again when a user denies system permission twice #17650
Comments
Hello @Amejia481 |
Thanks @VitalyVPinchuk for being interested in this issue, I was taking a closer look and I think it could take more work that I'm initially estimate, we prefer to work on the issue, please feel free to take other one, sorry for the inconveniences. |
This issue is not reproducible on Nightly 3/10, Beta 87.0.0-beta.4 from 3/8 and Release 86.1.1 from 2/23 with the following devices:
@Amejia481 Hi! Since the related issue mozilla-mobile/android-components#9812 is not fixed yet, I will move this in development. |
Thanks for testing, It could be that nightly version doesn't have the fix. Which AC version does it have? The version that have the fixed is ac 74.0.20210308190145 |
The Nightly tested has the AC version 74.0.20210309154120 and I could not reproduce the issue on this build, neither on the latest Beta or Release builds. |
Did you see the download dialog after denying permissions twice? |
Yes, the download dialog is displayed after denying the permissions twice. |
Sorry for the delay, the fixed is on nightly. The expected behavior is the dialog is shown when the permission is denied, indicating the user to go the settings page and allow the permission, if the user do so the download dialog will show the next time we try to download a file. screencapture-1615822262411.mp4 |
Verified as fixed on the 3/17 Nightly build, using a Google Pixel (Android 10). |
Thanks @lobontiumira ! |
If you try to download a file and deny the system dialog permission, and then do it again after that you are never going to get the system permission dialog again and all future downloads are going to be cancel by default without users noticing.
To fix that we have to do something similar to what we did on this pr, we have to use shouldShowRequestPermissionRationale to verify if permissions have been denied permanently and users have to go to the settings page and grant the permission there, for more information see https://developer.android.com/training/permissions/requesting#request-permission
┆Issue is synchronized with this Jira Task
The text was updated successfully, but these errors were encountered: