Skip to content

Android small icon support for unified notifications - #451

Open
aurimasc wants to merge 3 commits into
masterfrom
android-unified-small-icon
Open

Android small icon support for unified notifications#451
aurimasc wants to merge 3 commits into
masterfrom
android-unified-small-icon

Conversation

@aurimasc

@aurimasc aurimasc commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

https://jira.unity3d.com/browse/PLAT-21686
On Android notifications are required to have small icon. Unity APIs do not require it, we use app icon if not provided.
Unified notifications API does not allow to set icon as there is no such thing on iOS. However, there were user complains in the past that using something else than app icon is desirable. This PR adds ability to set small icon to NotificationCenter. This we we keep things the way they are - small icon cannot be set on notification, but user can configure the icon to be used on all notifications sent via unified APIs. If user wants to set icon per-notification, android specific notification API should be used instead of unified one.
Couple unrelated changes: one link in documentation does not work due to class not being fully qualified and a possible null exception was noticed for iOS (very low probability, only would happen in a scenario where notification would not work anyway due to not being setup properly).

Tested using Main test project (Unified scene). Icon is set up there and shows correctly. The scenario without icon is covered by automated tests.

@u-pr u-pr Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Harness Review

Needs changes

The change is small and focused, but the Android scheduling path now overrides notification state that existing unified callers may rely on preserving.

Reviewed commit 92d3bd6a

🤖 Helpful? 👍/👎

@aurimasc
aurimasc requested review from a team and manugildev July 28, 2026 11:45
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.

1 participant