feat: remove automatic APNs registration when requesting permissions (#15)

ameter · web-flow · commit 4932bd5199f1 · 2023-03-01T09:18:56.000-07:00
diff --git a/Sources/AmplifyUtilsNotifications/AUNotificationPermissions.swift b/Sources/AmplifyUtilsNotifications/AUNotificationPermissions.swift
@@ -44,11 +44,13 @@ public class AUNotificationPermissions {
             options: options
         )
         
-        if notificationsAllowed {
-            // Register with Apple Push Notification service
-            await Application.shared.registerForRemoteNotifications()
-        }
-        
         return notificationsAllowed
     }
+    
+    /// Register device with APNs
+    public static func registerForRemoteNotifications() async {
+        await MainActor.run {
+            Application.shared.registerForRemoteNotifications()
+        }
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -44,11 +44,13 @@ public class AUNotificationPermissions {`
`44`	`44`	`options: options`
`45`	`45`	`)`
`46`	`46`
`47`		`- if notificationsAllowed {`
`48`		`- // Register with Apple Push Notification service`
`49`		`- await Application.shared.registerForRemoteNotifications()`
`50`		`- }`
`51`		`-`
`52`	`47`	`return notificationsAllowed`
`53`	`48`	`}`
	`49`	`+`
	`50`	`+ /// Register device with APNs`
	`51`	`+ public static func registerForRemoteNotifications() async {`
	`52`	`+ await MainActor.run {`
	`53`	`+ Application.shared.registerForRemoteNotifications()`
	`54`	`+ }`
	`55`	`+ }`
`54`	`56`	`}`