From 66b5fad6c2768e82ddf02d50a26585f695bd274d Mon Sep 17 00:00:00 2001
From: Vitor Pamplona <vitor@vitorpamplona.com>
Date: Tue, 22 Aug 2023 11:23:11 -0400
Subject: [PATCH] Saves some loading time by moving the check if notification
 was already asked forward.

---
 .../amethyst/ui/screen/loggedIn/NotificationScreen.kt         | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/src/main/java/com/vitorpamplona/amethyst/ui/screen/loggedIn/NotificationScreen.kt b/app/src/main/java/com/vitorpamplona/amethyst/ui/screen/loggedIn/NotificationScreen.kt
index 645a61cc7..cba29abb1 100644
--- a/app/src/main/java/com/vitorpamplona/amethyst/ui/screen/loggedIn/NotificationScreen.kt
+++ b/app/src/main/java/com/vitorpamplona/amethyst/ui/screen/loggedIn/NotificationScreen.kt
@@ -113,12 +113,12 @@ var hasAlreadyAskedNotificationPermissions = false
 @OptIn(ExperimentalPermissionsApi::class)
 @Composable
 fun CheckifItNeedsToRequestNotificationPermission() {
-    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
+    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU && !hasAlreadyAskedNotificationPermissions) {
         val notificationPermissionState = rememberPermissionState(
             Manifest.permission.POST_NOTIFICATIONS
         )
 
-        if (!notificationPermissionState.status.isGranted && !hasAlreadyAskedNotificationPermissions) {
+        if (!notificationPermissionState.status.isGranted) {
             hasAlreadyAskedNotificationPermissions = true
 
             // This will pause the APP, including the connection with relays.