Page MenuHomePhabricator

Thanks extension should use new MediaWiki Notifications system instead of Echo
Closed, ResolvedPublic

Description

The Thanks extension uses the Echo to send thanks - (https://gerrit.wikimedia.org/g/mediawiki/extensions/Thanks/+/9c8c711616954442ca7acb53452edf6f79628f50/includes/Api/ApiCoreThank.php#247).

It's straightforward to trigger Thanks notification and the notification itself is one of the simpler ones. We should migrate the Thanks Notification to use the newly built MediaWiki Notifications system.

Let's focus only on Core notification, no need to update flow notification as Flow is currently deprecated. (we've done them both)

Acceptance criteria:

  • when Echo is not present, MediaWiki should send an email with Thanks notification (cancelled per T383995)
  • when Echo is present, there is no visible behaviour to users

Note:
This task is strictly about triggering Thanks notification and it's to verify new MediaWiki notifications (and Echo provider) behaviour.

Event Timeline

pmiazga renamed this task from Use new MediaWiki Notifications system instead of Echo to Thanks extension should use new MediaWiki Notifications system instead of Echo.Jan 30 2025, 4:57 PM
pmiazga updated the task description. (Show Details)

Change #1108513 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):

[mediawiki/extensions/Thanks@master] [WIP] Notifications in core

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

Change #1108513 merged by jenkins-bot:

[mediawiki/extensions/Thanks@master] Use MediaWiki's built-in notification system for thanks events

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

matmarex updated the task description. (Show Details)