Opened 5 years ago

Last modified 4 months ago

#2386 closed change

Make it possible to register a notification download listener — at Version 3

Reported by: fhd Assignee:
Priority: P2 Milestone:
Module: Core Keywords: closed-in-favor-of-gitlab
Cc: sebastian Blocked By:
Blocking: #2367, #2368, #2387 Platform: Unknown
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by trev)

Background

In #2366, we want to react to notification downloads - there's currently no non-hacky way to do that.

What to change

Add API for registering a notification download listener:

  • Notification.addNewNotificationListener()
  • Notification.removeNewNotificationListener()

The listeners should be triggered after downloading notifications but only if that download changed the result that Notification.getNextToShow() would return.

Change History (3)

comment:1 Changed 5 years ago by fhd

  • Cc trev added

Wladimir: Not set to ready yet, please have a look first.

comment:2 Changed 5 years ago by fhd

  • Blocking 2387 added

comment:3 Changed 5 years ago by trev

  • Description modified (diff)
  • Ready set
Note: See TracTickets for help on using tickets.