Skip to content

Commit 6ab28a9

Browse files
authored
Merge pull request #1615 from OneSignal/fix/notification_prompt_callback_firing_on_foregrounding
[Fix] notification permission prompt callback firing when app is foregrounded
2 parents fadcee4 + d5614ce commit 6ab28a9

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

OneSignalSDK/onesignal/src/main/java/com/onesignal/NotificationPermissionController.kt

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@ object NotificationPermissionController : PermissionsActivity.PermissionCallback
3636

3737
private val callbacks:
3838
MutableSet<OneSignal.PromptForPushNotificationPermissionResponseHandler> = HashSet()
39+
private var awaitingForReturnFromSystemSettings = false
3940

4041
init {
4142
PermissionsActivity.registerAsCallback(PERMISSION_TYPE, this)
@@ -97,6 +98,7 @@ object NotificationPermissionController : PermissionsActivity.PermissionCallback
9798
object : AlertDialogPrepromptForAndroidSettings.Callback {
9899
override fun onAccept() {
99100
NavigateToAndroidSettingsForNotifications.show(activity)
101+
awaitingForReturnFromSystemSettings = true
100102
}
101103
override fun onDecline() {
102104
fireCallBacks(false)
@@ -113,6 +115,8 @@ object NotificationPermissionController : PermissionsActivity.PermissionCallback
113115
}
114116

115117
fun onAppForegrounded() {
118+
if (!awaitingForReturnFromSystemSettings) return
119+
awaitingForReturnFromSystemSettings = false
116120
fireCallBacks(notificationsEnabled())
117121
}
118122

0 commit comments

Comments
 (0)