Page MenuHomePhabricator
Create Task
Maniphest T125428

Make 'flow-mention' more important than other notifications
Closed, ResolvedPublic
Actions

Description

When an event results in more than one notifications and one of those notifications is a mention (or a flow-mention), deliver the mention and suppress everything else.

Details

Related Changes in Gerrit:
SubjectAuthorRepoBranchLines +/-
Mentions are the primary notificationMatthias Mulliemediawiki/extensions/Flowmaster+29 -41
Allow certain users to be excludedMatthias Mulliemediawiki/extensions/Echomaster+41 -24
Customize query in gerrit

Related Objects

Event Timeline

SBisson created this task.Feb 1 2016, 7:36 PM
SBisson raised the priority of this task from to Needs Triage.
SBisson updated the task description. (Show Details)
SBisson added projects: StructuredDiscussions, Collaboration-Team-Archive-2015-2016.
SBisson added subscribers: SBisson, StudiesWorld, Aklapper.
SBisson mentioned this in T125121: Re-evaluate Flow user locators (who gets notified) for notifications.Feb 1 2016, 7:43 PM
gerritbot subscribed.Feb 2 2016, 1:21 PM

Change 267876 had a related patch set uploaded (by Matthias Mullie):
Allow certain users to be excluded

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

gerritbot added a project: Patch-For-Review.Feb 2 2016, 1:21 PM
gerritbot added a comment.Feb 2 2016, 1:24 PM

Change 267877 had a related patch set uploaded (by Matthias Mullie):
Mentions are the primary notification

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

matthiasmullie claimed this task.Feb 2 2016, 1:24 PM
matthiasmullie moved this task from Untriaged to Needs Review on the Collaboration-Team-Archive-2015-2016 board.
matthiasmullie set Security to None.
SBisson updated the task description. (Show Details)Feb 2 2016, 1:29 PM
SBisson moved this task from Needs Review to Code Review Started on the Collaboration-Team-Archive-2015-2016 board.Feb 2 2016, 1:37 PM
SBisson moved this task from Code Review Started to Needs Review on the Collaboration-Team-Archive-2015-2016 board.Feb 2 2016, 3:20 PM
SBisson moved this task from Needs Review to Code Review Started on the Collaboration-Team-Archive-2015-2016 board.Feb 4 2016, 6:29 PM
gerritbot added a comment.Feb 4 2016, 7:24 PM

Change 267876 merged by jenkins-bot:
Allow certain users to be excluded

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

SBisson moved this task from Code Review Started to QA Review on the Collaboration-Team-Archive-2015-2016 board.Feb 4 2016, 7:27 PM
gerritbot added a comment.Feb 4 2016, 7:36 PM

Change 267877 merged by jenkins-bot:
Mentions are the primary notification

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

ReleaseTaggerBot added a project: MW-1.27-release (WMF-deploy-2016-02-09_(1.27.0-wmf.13)).Feb 4 2016, 8:06 PM
Etonkovidova subscribed.Feb 8 2016, 5:43 PM

Checked for

  • local cross-wiki notifications
  • watched/non watched pages and topics
  • whether the other notifications were suppressed - e.g. Mention in newly created topic/posts or in edited posts

Mention notifications come are prevalent notification type for the above cases - other notifications are suppressed.

Note: Still do not see Mention notifications from

  • Create/edit a Summary
  • Board descritpion
Etonkovidova moved this task from QA Review to Product Review on the Collaboration-Team-Archive-2015-2016 board.Feb 8 2016, 5:44 PM
• jmatazzoni subscribed.Feb 17 2016, 11:21 PM

@Etonkovidova, you note that you:

Still do not see Mention notifications from

  • Create/edit a Summary
  • Board description

Closing this because those are in T125120

• jmatazzoni closed this task as Resolved.Feb 17 2016, 11:21 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits