Skip to content

Additional notification code snippets migration#929

Open
alabiaga wants to merge 3 commits into
android:mainfrom
alabiaga:notification_snippets
Open

Additional notification code snippets migration#929
alabiaga wants to merge 3 commits into
android:mainfrom
alabiaga:notification_snippets

Conversation

@alabiaga
Copy link
Copy Markdown
Member

Change-Id: Ib7b51173d55bf49626fe9b19460e4d04d6994470

For documentation in: https://developer.android.com/develop/ui/compose/notifications/create-notification

Change-Id: Ib7b51173d55bf49626fe9b19460e4d04d6994470
@alabiaga alabiaga requested a review from a team as a code owner May 22, 2026 19:03
@alabiaga alabiaga requested a review from raystatic May 22, 2026 19:03
@snippet-bot
Copy link
Copy Markdown

snippet-bot Bot commented May 22, 2026

Here is the summary of changes.

You are about to add 15 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces comprehensive notification snippets, including channel creation, basic and styled notifications, tap actions, and reply handling with a dedicated ReplyReceiver. Feedback focuses on addressing a potential resource leak by properly closing an InputStream in ReplyReceiver, improving type safety when accessing the NotificationManager, and using shared constants for notification reply keys to ensure consistency.

alabiaga added 2 commits May 22, 2026 14:15
Change-Id: Ia2c1c24f4c11b8273210a641485c90fb17985c54
Change-Id: If7261d0c60c902ba0a3b59043c3ce94d9c2c2390
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