Skip to content

Permission Push#884

Open
Appteamit wants to merge 2 commits intoXtraball:release/5.0.17from
Appteamit:trunk
Open

Permission Push#884
Appteamit wants to merge 2 commits intoXtraball:release/5.0.17from
Appteamit:trunk

Conversation

@Appteamit
Copy link

@Appteamit Appteamit commented Oct 9, 2025

I've fixed the stuck startup issue and now forward diagnostics to the osAppId.

  • I've fixed the TypeError: HASH_ON_START.match is not a function error by setting HASH_ON_START to a string, not a Boolean.
  • I've added shims for OneSignal.Notifications.addPermissionObserver in addition to the existing ones for getPermission, canRequestPermission, and requestPermission, avoiding additional TypeErrors.
  • I've introduced an optional fallback: if you define window.OS_APP_ID in index.html, Push2Service bootstrap will use it to initialize OneSignal even if Application.application is not yet populated

index.html

Add inside manifest.xml
Runtime notification permission required from Android 13 (API 33) and above
<uses-permission android:name="android.permission.POST_NOTIFICATIONS" />

Immagine WhatsApp 2025-10-09 ore 22 04 35_90fb05ba

Video.WhatsApp.2025-10-09.ore.22.12.36_2cb5f537.mp4

@Neustradamus
Copy link

@Xtraball team, @Slicit, @alexyves: Have you seen this PR?

@rogerdavis198
Copy link

update I have added

to my manifest file and the notifications work now , just uploading the updated app to see if it goes live or errors out in play store .
thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants