Summary
'Groups' was ranked the third most important piece of information in the feedback survey that we ran.
Acceptance criteria
- List of explicit groups (not implicit groups like *, user, temp) appear in the user info card
Description
Description
Details
Details
Related Changes in Gerrit:
| Subject | Author | Repo | Branch | Lines +/- | |
|---|---|---|---|---|---|
| UserInfoCard: Display user groups | Kosta Harlan | mediawiki/extensions/CheckUser | master | +59 -12 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | KColeman-WMF | T384725 [Epic] WE4.2.18 UserInfoCard experiment for assisting patrollers and moderators | |||
| Resolved | KColeman-WMF | T384702 UserInfoCard: Design a clickable component to display account information | |||
| Resolved | kostajh | T391831 UserInfoCard: Display user groups |
Event Timeline
Comment Actions
@KColeman-WMF should this be "Groups:" for the label and a comma-separated list of groups?
Comment Actions
Change #1160879 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):
[mediawiki/extensions/CheckUser@master] (WIP) UserInfoCard: Display groups
Comment Actions
Yes, in the mockup it says Groups: autopatrolled, event coordinator.
Should the group name be the name used in code, or the human-readable value? e.g. for event coordinator:
- machine name: eventcoordinator
- translated label: Event coordinators, Coordinadores de eventos (Spanish)
The translated version is generally ideal, but the machine name is going to be much shorter (albeit in English only)
Comment Actions
Yes, in the mockup it says Groups: autopatrolled, event coordinator.
Should the group name be the name used in code, or the human-readable value? e.g. for event coordinator:
- machine name: eventcoordinator
- translated label: Event coordinators, Coordinadores de eventos (Spanish)
The translated version is generally ideal, but the machine name is going to be much shorter (albeit in English only)
Having written that out, I think we need to go with the translated labels associated with these groups.
Comment Actions
Yes, in the mockup it says Groups: autopatrolled, event coordinator.
Should the group name be the name used in code, or the human-readable value? e.g. for event coordinator:
- machine name: eventcoordinator
- translated label: Event coordinators, Coordinadores de eventos (Spanish)
The translated version is generally ideal, but the machine name is going to be much shorter (albeit in English only)
Having written that out, I think we need to go with the translated labels associated with these groups.
Yes I agree with that.
Comment Actions
Change #1160879 merged by jenkins-bot:
[mediawiki/extensions/CheckUser@master] UserInfoCard: Display user groups
Comment Actions
QA is completed: The new code has been implemented and is functioning as expected (Per the Acceptance Criteria).
