Page MenuHomePhabricator

"mention" notifications on Meta don't work for ns0
Closed, ResolvedPublic

Description

"mention" notifications (one's userpage being linked) on Meta don't work for the main namespace, which however hosts some important discussion and request pages there.


Version: unspecified
Severity: normal
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=54433

Details

Reference
bz55491

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 2:18 AM
bzimport set Reference to bz55491.

The issue here is that it's hardcoded for talk namespaces and NS_PROJECT. We probably need some kind of configuration variable to modify it.

Echo/Hooks.php:455 if ( $title->isTalkPage() || $title->inNamespace( NS_PROJECT ) ) {

(In reply to comment #1)

The issue here is that it's hardcoded for talk namespaces and NS_PROJECT. We
probably need some kind of configuration variable to modify it.

Echo/Hooks.php:455 if ( $title->isTalkPage() ||
$title->inNamespace( NS_PROJECT ) ) {

Is there any reason it needs to be restricted to certain namespaces at all?

The WMF core features team tracks this bug on Mingle card https://mingle.corp.wikimedia.org/projects/flow/cards/326, but people from the community are welcome to contribute here and in Gerrit.

Change 93508 had a related patch set uploaded by Gerrit Patch Uploader:
Remove namespace restriction for mention notifications

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

  • Bug 56919 has been marked as a duplicate of this bug. ***

Just to note that there is a need for notifications in the template namespace as well. Wikipedia DYK makes extensive use of this namespace for reviews.

@Spinningspark the current patch address the notifications triggered by a signature.

Previous behavior: only on NS_PROJECT (Wikipedia:) and on every talk namespaces.
Patch behavior: everywhere

I offered in comments to make this configurable.

If the patch is merged as is, template namespace will be covered.
If my solution is used, the following configuration will be needed:

$wgEchoNotifications['baseNamespaces'] = array(

default => array(
    NS_PROJECT
),

metawiki => array(
    NS_PROJECT,
    NS_MAIN, //bug 55491
),

enwiki => array(
    NS_PROJECT,
    NS_TEMPLATE, //bug 55491
),

);

I don't see any reason why it should be made configurable. Why would we not want notifications from particular namespaces? If my name were to be linked in article space I would certainly want to know about it top priority above everything else. Perhaps that case could be implemented as a big orange bar at the top of the page (alright, perhaps not :) ).

(In reply to comment #8)

I don't see any reason why it should be made configurable. Why would we not
want notifications from particular namespaces?

Edge cases, I imagine. But we should test it out and see what happens, in my opinion.

(In reply to comment #9)

(In reply to comment #8)

I don't see any reason why it should be made configurable. Why would we not
want notifications from particular namespaces?

Edge cases, I imagine. But we should test it out and see what happens, in my
opinion.

Edge cases are likely to be specific pages rather than entire namespaces.

(In reply to comment #10)

Edge cases are likely to be specific pages rather than entire namespaces.

Sure, but... bug 50628. :-)

What is hindering the approval of this patch? I see more and more that people are trying to ping each other on the main or project ns'es.

(In reply to comment #12)

What is hindering the approval of this patch? I see more and more that people
are trying to ping each other on the main or project ns'es.

https://gerrit.wikimedia.org/r/93508 needs to be updated and merged. I just bumped the changeset.

shake shake shake shake shake. One month later since last shaking

Change 93508 merged by jenkins-bot:
Remove namespace restriction for mention notifications

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