Camera permission rationale #11520
Camera permission rationale #11520
Conversation
|
stable-IT test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/8280-IT-stable-15-03 |
|
master-IT test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/8280-IT-master-15-30 |
| @@ -443,6 +443,9 @@ public void onRequestPermissionsResult(int requestCode, @NonNull String[] permis | |||
| // permission was granted | |||
| getFileOperationsHelper() | |||
| .uploadFromCamera(this, FileDisplayActivity.REQUEST_CODE__UPLOAD_FROM_CAMERA); | |||
| } else if (!shouldShowRequestPermissionRationale(permissions[0])) { | |||
| // user CHECKED "never ask again" | |||
| DisplayUtils.showSnackMessage(this, R.string.camera_permission_rationale); | |||
There was a problem hiding this comment.
With which Android did you test this?
With Android 10 I only have "allow" or "deny"
There was a problem hiding this comment.
Check with Android 9.
There is radio button for "Don't ask again".
There was a problem hiding this comment.
Tested with Android 9 emulator.
Maybe this is vendor specific?
There was a problem hiding this comment.
Yes that can might be the case because we have found this issue in Huawei device.
The issue will only occur in devices where there is option to deny the permission by checking "Don't ask again" (for lower os versions) or "Don't allow" (for OS 11+) devices.
In both the cases no error snackbar shown to the user.
For more test you can check on Android 11+ devices and select "Don't allow". And again if you click on camera option then no permission dialog and message will be shown.
surinder-tsys
force-pushed
the
nmc/1941-cameraPermissionRationale
branch
from
d4fec26
to
917bd5d
Compare
surinder-tsys
force-pushed
the
nmc/1941-cameraPermissionRationale
branch
from
917bd5d
to
3c4dbb6
Compare
|
@tobiasKaminsky @surinder-tsys I tested with Android 14 emulator. After tap the don't allow, user already not seeing any error message. android_14.mov |
surinder-tsys
force-pushed
the
nmc/1941-cameraPermissionRationale
branch
from
3c4dbb6
to
8822573
Compare
surinder-tsys
force-pushed
the
nmc/1941-cameraPermissionRationale
branch
from
8822573
to
7443459
Compare
surinder-tsys
force-pushed
the
nmc/1941-cameraPermissionRationale
branch
2 times, most recently
from
0b30011
to
8be45e3
Compare
surinder-tsys
force-pushed
the
nmc/1941-cameraPermissionRationale
branch
from
8be45e3
to
a1fe95c
Compare
|
This PR is no longer applicable. So closing the PR. |
Pre condition:
App camera permission should not enabled
Step to reproduce:
Actual result:
After allow permission applied successfully but error message coming to set permission manually.
Expected result:
Error message should not come if permission set successfully