Page MenuHomePhabricator

notification-header-user-rights-* should have GENDER support for the current user
Closed, ResolvedPublic

Description

The following messages should have GENDER support for the current user:

  • notification-header-user-rights-add-only
  • notification-header-user-rights-remove-only
  • notification-header-user-rights-add-and-remove

Now they seem to have support only for the gender of the performer of the action.

Details

Related Gerrit Patches:

Event Timeline

Amire80 created this task.Jan 29 2016, 8:52 PM
Amire80 raised the priority of this task from to Needs Triage.
Amire80 updated the task description. (Show Details)
Amire80 added projects: Notifications, I18n.
Amire80 added subscribers: Amire80, Mooeypoo.
Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptJan 29 2016, 8:52 PM
Restricted Application added subscribers: StudiesWorld, Aklapper. · View Herald Transcript
SBisson claimed this task.Jan 29 2016, 9:08 PM
SBisson triaged this task as Medium priority.
SBisson set Security to None.

Change 267325 had a related patch set uploaded (by Sbisson):
Gender support for viewing user in user-rights notifications

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

Change 267325 merged by jenkins-bot:
Gender support for viewing user in user-rights notifications

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

Checked in betalabs (thanks to @Catrope again) - the same verifying method as in T125253: Notification-header-flow-topic-renamed-user-talk and Notification-bundle-header-flow-post-edited-user-talk seem to have incorrect parameter documentation.

Gender support is added for a current user (just samples):

jmatazzoni closed this task as Resolved.Mar 7 2016, 10:11 PM