Skip to content

hotfix/cp-10693-android-removenotification-is-not-working-as-expected#329

Merged
gbhrdt-cp merged 1 commit intomasterfrom
hotfix/cp-10693-android-removenotification-is-not-working-as-expected
Jan 20, 2026
Merged

hotfix/cp-10693-android-removenotification-is-not-working-as-expected#329
gbhrdt-cp merged 1 commit intomasterfrom
hotfix/cp-10693-android-removenotification-is-not-working-as-expected

Conversation

@unnaticleverpush
Copy link
Contributor

@unnaticleverpush unnaticleverpush commented Jan 19, 2026

Fix removeNotification to also clear the group’s summary notification.


Note

Fixes notification removal to also clean up group summaries when the last child is deleted.

  • Enhances removeNotification(...) to derive a groupKey from the notification’s category and pass it to removeGroupSummaries(...)
  • Adds removeGroupSummaries(...) (Android M+) to count active child notifications via BadgeHelper.getActiveNotifications(...) and cancel a group’s summary when no children remain; skips the just-removed child and optionally filters by the provided group key
  • Imports StatusBarNotification and adds error handling/logging around cleanup

Written by Cursor Bugbot for commit 0ce811b. This will update automatically on new commits. Configure here.


Summary by cubic

Fixes removeNotification on Android to clear the group summary when its last child is removed. Addresses CP-10693 and prevents stale summary notifications.

  • Bug Fixes
    • Derives the group key from the notification category and tracks the removed child’s tag/id.
    • Counts active child notifications via StatusBarNotification, skipping the just-removed child.
    • Cancels the group summary when no children remain (Android M+).

Written for commit 0ce811b. Summary will update on new commits.

Fix removeNotification to also clear the group’s summary notification.
@unnaticleverpush unnaticleverpush self-assigned this Jan 19, 2026
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@gbhrdt-cp gbhrdt-cp merged commit f5c47b8 into master Jan 20, 2026
4 of 7 checks passed
@gbhrdt-cp gbhrdt-cp deleted the hotfix/cp-10693-android-removenotification-is-not-working-as-expected branch January 20, 2026 10:46
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.

3 participants