Page MenuHomePhabricator

Migrate Enotif to the new notifications interface
Closed, ResolvedPublic

Description

As part of the work on "simplify feature development" (KR 5.2) we want to consolidate the operation of notifications through the new core notification system and interface.

This means that "Enotif" watchlist notifications should pass through the same pathway and use the same interface as notifications stemming from Echo/Notifications extension and use the same workflow.

At the moment, Enotif notifications will be set to be handled by a separate Handler, so the user-behavior in terms of enotif notifications remains the same. However, this will hopefully be a temporary Handler, as the change opens the door to have the main notifications Handler (Echo notifications extension, at the moment) handle those notifications as well.

This brings us closer to resolving a long-standing developer conversation regarding merging notifications into Core and improving the maintenance, and the ability for feature teams to add and manage notifications across the system.

Event Timeline

JTweed-WMF triaged this task as Low priority.

Change #1137458 had a related patch set uploaded (by Pmiazga; author: Pmiazga):

[mediawiki/core@master] Enotif: Utility methods to verify Notification source

https://gerrit.wikimedia.org/r/1137458

Change #1137460 had a related patch set uploaded (by Pmiazga; author: Pmiazga):

[mediawiki/core@master] Notifications: Introduce Filter middleware

https://gerrit.wikimedia.org/r/1137460

Change #1137460 merged by jenkins-bot:

[mediawiki/core@master] Notifications: Introduce Filter middleware

https://gerrit.wikimedia.org/r/1137460

Change #1137458 merged by jenkins-bot:

[mediawiki/core@master] Notifications: Utility methods to verify Notification source

https://gerrit.wikimedia.org/r/1137458

Change #1143822 had a related patch set uploaded (by Pmiazga; author: Pmiazga):

[mediawiki/core@master] enotif: Talk notification do not require confirmed email

https://gerrit.wikimedia.org/r/1143822

Change #1143822 merged by jenkins-bot:

[mediawiki/core@master] enotif: Talk notification do not require confirmed email

https://gerrit.wikimedia.org/r/1143822

matmarex claimed this task.