Page MenuHomePhabricator

add Flow support to TranslationNotifications
Closed, DuplicatePublic

Description

aharoni hi dannyh
dannyh ebernhardson superm401 : Amir has a question -- if he enabled Flow on his user talk page on hewiki, what would happen if MassMessage tried to write on his page?
dannyh ah, and here he is :)
legoktm dannyh: it would just work
aharoni !
aharoni legoktm: great to hear
legoktm MassMessage has special Flow and LQT support
dannyh oh, excellent -- I didn't realize.
dannyh thanks!
aharoni Now I wonder about TranslationNotifications, which I made in 2012 and barely touched since then.
legoktm TN would not work :P
aharoni legoktm: from where can I copypaste this to TranslationNotifications?
legoktm aharoni: https://github.com/wikimedia/mediawiki-extensions-MassMessage/blob/master/includes/job/MassMessageJob.php#L199
aharoni legoktm: excellent, thanks a lot.
legoktm and https://github.com/wikimedia/mediawiki-extensions-MassMessage/blob/master/includes/job/MassMessageJob.php#L152
legoktm :)

Event Timeline

Amire80 raised the priority of this task from to Low.
Amire80 updated the task description. (Show Details)
Amire80 subscribed.

MassMessage was based off TranslationNotifications, it sounds like the job now is to remove the messaging part of TranslationNotifications and use MassMessage for that?

MassMessage was based off TranslationNotifications, it sounds like the job now is to remove the messaging part of TranslationNotifications and use MassMessage for that?

Maybe, though we'll have to be careful about not breaking current features. The first thing I can think of is that TN's subscribers list is private and based on user preferences, and MM is public. But yeah, this may be doable.