Merge pull request orgzly-revived#245 from amberin/git-via-ssh-requir…

…es-notification-permission Ensure notifications permission is granted before using Git via SSH
amberin · May 10, 2024 · e2f2091 · e2f2091
2 parents a4a8e72 + a831015
commit e2f2091
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/app/src/main/java/com/orgzly/android/ui/repo/git/GitRepoActivity.kt b/app/src/main/java/com/orgzly/android/ui/repo/git/GitRepoActivity.kt
@@ -208,6 +208,13 @@ class GitRepoActivity : CommonActivity(), GitPreferences {
             binding.activityRepoGitHttpsAuthInfo.visibility = View.GONE
             binding.activityRepoGitHttpsUsernameLayout.visibility = View.GONE
             binding.activityRepoGitHttpsPasswordLayout.visibility = View.GONE
+            // Using SSH transport requires notification permission (for server key verification)
+            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
+                AppPermissions.isGrantedOrRequest(
+                    App.getCurrentActivity(),
+                    AppPermissions.Usage.POST_NOTIFICATIONS
+                )
+            }
         }
     }
 
@@ -513,7 +520,6 @@ class GitRepoActivity : CommonActivity(), GitPreferences {
             fragment.repoCheckComplete(e)
         }
 
-
         override fun start(totalTasks: Int) {
             publishProgress(CloneProgressUpdate(totalTasks, true))
         }