Closed Bug 1909633 Opened 6 months ago Closed 6 months ago

Updating cached in-app notifications pushes unfiltered list to NotificationManager

Categories

(Thunderbird :: General, defect)

Product:
Thunderbird
Component:
General
Version:
Thunderbird 128
Type:
defect

Tracking

(thunderbird_esr128 affected, thunderbird129 affected, thunderbird130 fixed)

Status:
RESOLVED FIXED
Milestone:
130 Branch
Tracking Flags:
Tracking Status
thunderbird_esr128 --- affected
thunderbird129 --- affected
thunderbird130 --- fixed

People

(Reporter: freaktechnik, Assigned: freaktechnik)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Bug 1909633 - Pass filtered list of notifications to NotificationManager on update. r=#thunderbird-reviewers
48 bytes, text/x-phabricator-request
Details | Review

When we cache a new list of notifications, we should only push a filtered list of notifications to the NotificationManager. It might make sense to consolidate the calls for that into a pseudo-private method, so we don't do the same thing in three call sites.

Version: unspecified → Thunderbird 128
Assignee: nobody → martin
Status: NEW → ASSIGNED
status-thunderbird129: --- → affected
status-thunderbird130: --- → fixed
status-thunderbird_esr128: --- → affected
Keywords: checkin-needed-tb
Target Milestone: --- → 130 Branch

Pushed by alessandro@thunderbird.net:
https://hg.mozilla.org/comm-central/rev/ccb97319a48c
Pass filtered list of notifications to NotificationManager on update. r=aleca

Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Keywords: checkin-needed-tb
Resolution: --- → FIXED
Blocks: tb128found
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: