Page MenuHomePhabricator

Add/Remove/Notify of Publishers should happen in bulk
Closed, ResolvedPublic

Description should accept UserIdList just like addSubscription above. Same for RemovePublishers.

Possibly affected code would be:

  1. -- here we should directly call our addSubscription with the whole list of users instead.
  2. Below that, the notifyPublishers() should also take a userIdList and this should happen in the next line.
  3. -- same here.
  4. Probably the removePublishers() too.

Difficulty: Moderately tough, but easy once you read the examples given above.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptDec 24 2017, 12:11 PM

Change 400464 had a related patch set uploaded (by Pppery; owner: Pppery):
[mediawiki/extensions/Newsletter@master] Publisher add/remove functions now take user ids array

Change 400464 merged by jenkins-bot:
[mediawiki/extensions/Newsletter@master] Publisher add/remove functions now take user ids array

01tonythomas closed this task as Resolved.Dec 29 2017, 9:18 PM
01tonythomas assigned this task to Pppery.

Thanks :)