Page MenuHomePhabricator

Need support for Echo notification preferences
Closed, ResolvedPublic3 Estimated Story Points

Description

As established in T172083: Allow timezone to be global, Echo notification checkboxes (field type HTMLCheckMatrix) aren't currently globalizable.

Снимок экрана 2017-08-04 в 19.10.07.png (505×396 px, 55 KB)

We'll do one global preference for a set of notification preferences. If people want to use a local exception for notifications, then they'll start with the default on that local wiki, and make changes from there.

Event Timeline

kaldari renamed this task from Need support for Echo notification preferences to Need support for Echo notification preferences .Aug 22 2017, 11:33 PM
kaldari updated the task description. (Show Details)

@DannyH: We need a product decision on this. Do we want one checkbox for the entire matrix, one checkbox per row, or one checkbox per option?

We should do this all as one chunk. An extra column of checkboxes, plus allowing for local exceptions, would make this insane.

We'll do one global preference for a set of notification preferences. If people want to use a local exception for notifications, then they'll start with the default on that local wiki, and make changes from there.

Just to clarify this means that the entire HTMLCheckMatrix will have 1 checkbox for making the entire matrix global.

kaldari triaged this task as Medium priority.Sep 5 2017, 11:14 PM
kaldari set the point value for this task to 3.

Just to clarify this means that the entire HTMLCheckMatrix will have 1 checkbox for making the entire matrix global.

Are you sure this is a good idea? An addition column in the matrix for global checks don't seem that complicated. An idea would be to depart from the standard approach at any HTMLCheckMatrix and to add such a col instead.

Notification settings can be considered as part of the settings that are in particulary many cases changed by many users, so I think a single checkbox doesn't do it justice.

Change 378843 had a related patch set uploaded (by Samwilson; owner: Samwilson):
[mediawiki/core@master] Add classes to HTMLCheckMatrix items to identify forced ones

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

Change 378843 merged by jenkins-bot:
[mediawiki/core@master] Add classes to HTMLCheckMatrix items to identify forced ones

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

Change 401893 had a related patch set uploaded (by Samwilson; owner: Samwilson):
[mediawiki/extensions/GlobalPreferences@master] Handle CheckMatrix fields

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

Change 401893 merged by jenkins-bot:
[mediawiki/extensions/GlobalPreferences@master] Handle CheckMatrix fields

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