Merge pull request #1783 from FelixZY/fzy/1781/1

[Permissions] fix(gh-1781): check for `Denied(shouldShowRationale=false)` in `MutableMultiplePermissionsState.shouldShowRationale`
google · Nov 25, 2024 · b019980 · b019980
2 parents 52a660d + b80c533
commit b019980
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/...sions/src/main/java/com/google/accompanist/permissions/MutableMultiplePermissionsState.kt b/...sions/src/main/java/com/google/accompanist/permissions/MutableMultiplePermissionsState.kt
@@ -127,7 +127,8 @@ internal class MutableMultiplePermissionsState(
     }
 
     override val shouldShowRationale: Boolean by derivedStateOf {
-        permissions.any { it.status.shouldShowRationale }
+        permissions.any { it.status.shouldShowRationale } &&
+            permissions.none { !it.status.isGranted && !it.status.shouldShowRationale }
     }
 
     override fun launchMultiplePermissionRequest() {