Don't change setting to don't ask again unless that's what we've got from system

aurimasc · aurimasc · commit 76d748f89bf5 · 2022-09-19T17:23:59.000+03:00
diff --git a/com.unity.mobile.notifications/Runtime/Android/AndroidNotificationCenter.cs b/com.unity.mobile.notifications/Runtime/Android/AndroidNotificationCenter.cs
@@ -686,7 +686,7 @@ public static PermissionStatus RequestPermissionToPost()
             // Can only request permission if applications target SDK is 33, not actual device SDK
             if (s_TargetApiLevel < API_POST_NOTIFICATIONS_PERMISSION_REQUIRED)
             {
-                SetPostPermissionSetting(PermissionStatus.DeniedDontAskAgain);
+                // don't change setting here, in case app gets updated
                 return PermissionStatus.DeniedDontAskAgain;
             }
 

Original file line number	Diff line number	Diff line change
`@@ -686,7 +686,7 @@ public static PermissionStatus RequestPermissionToPost()`
`686`	`686`	`// Can only request permission if applications target SDK is 33, not actual device SDK`
`687`	`687`	`if (s_TargetApiLevel < API_POST_NOTIFICATIONS_PERMISSION_REQUIRED)`
`688`	`688`	`{`
`689`		`- SetPostPermissionSetting(PermissionStatus.DeniedDontAskAgain);`
	`689`	`+ // don't change setting here, in case app gets updated`
`690`	`690`	`return PermissionStatus.DeniedDontAskAgain;`
`691`	`691`	`}`
`692`	`692`