Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Invalid notification (no valid small icon) on some devices #550

Closed
1 task done
MaxBaranuk opened this issue Nov 7, 2022 · 1 comment
Closed
1 task done

Comments

@MaxBaranuk
Copy link

MaxBaranuk commented Nov 7, 2022

What happened?

Lots of crash reports in analytics with such stack trace:

Fatal Exception: java.lang.RuntimeException: Unable to start receiver com.DgnGames.AndroidNotifications.UnityNotificationManager: java.lang.IllegalArgumentException: Invalid notification (no valid small icon)

Mostly on Samsung, but there are some other devices present (Amazon, Motorolla)
Different Android versions

Steps to reproduce?

We can not reproduce this on any of our test devices. Pushes are received successfully, and no crashes are found.

What did you expect to happen?

Try to understand the reason of crashes

Unity version

2021.3.4f1

OneSignal Unity SDK version

3.0.2

Platform

Android

Relevant log output

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@jkasten2
Copy link
Member

@MaxBaranuk Thanks for reporting, based on the error you submitted it looks be an issue with your com.DgnGames.AndroidNotifications.UnityNotificationManager Java class:

Fatal Exception: java.lang.RuntimeException: Unable to start receiver com.DgnGames.AndroidNotifications.UnityNotificationManager: java.lang.IllegalArgumentException: Invalid notification (no valid small icon)

OneSignal has its own push handler for Android so you don't need your own. If you don't need to process pushes from any other source than OneSignal you can remove push handling from your DgnGames.AndroidNotifications.UnityNotificationManager class. If you do remove it make sure to remove your AndroidManifest.xml entry too.

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

No branches or pull requests

3 participants