Page MenuHomePhabricator

All notifications in preferences should check all notification rows JS-side
Closed, ResolvedPublic

Description

If you go to http://localhost:8000/my/preferences/ and check "All notifications", it won't check all rows.

Backend pretends they are (so if you save it, all rows will be checked by backend - and backend will ignore the fact they're checked, just store the first one), so this just need to be done by JS too, so it'll look better to the user.

Event Timeline

Change 480453 had a related patch set uploaded (by Nathan fraignt; owner: Nathan fraignt):
[wikimedia-cz/tracker@master] All notifications input checks all notification rows

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

Change 480453 merged by jenkins-bot:
[wikimedia-cz/tracker@master] All notifications input checks all notification rows

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