[native] avoid parsing badge-only notifs
ClosedPublic
Actions

Authored by kamil on Jun 26 2025, 8:20 AM.

Details

Reviewers

tomek
angelika

Commits

rCOMM865d054d8abe: [native] avoid parsing badge-only notifs

Summary

ENG-10959.

Test Plan

There is no confusing error log described in task
Unread count works when app is in both background and foreground
Notifs work for both thin and thick threads

Diff Detail

Repository

rCOMM Comm

Branch

fix-log

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Jun 26 2025, 8:20 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptJun 26 2025, 8:20 AM

kamil published this revision for review.Jun 26 2025, 8:22 AM

tomek accepted this revision.Jun 26 2025, 8:45 AM

This revision is now accepted and ready to land.Jun 26 2025, 8:45 AM

Harbormaster completed remote builds in B34463: Diff 48579.Jun 26 2025, 8:53 AM

Closed by commit rCOMM865d054d8abe: [native] avoid parsing badge-only notifs. · Explain WhyJun 26 2025, 9:04 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM865d054d8abe: [native] avoid parsing badge-only notifs.

Revision Contents
Changeset List

Path

Size

native/

ios/

Comm/

CommIOSNotifications/

CommIOSNotifications.mm

5 lines

Diff 48579

View Options

native/ios/Comm/CommIOSNotifications/CommIOSNotifications.mm

[native] avoid parsing badge-only notifsClosedPublicActions