Skip to content

Add Reviewer name to sidebar Reviewers list

Thomas Randolph requested to merge tor/feature/reviewer-full-names into master

What does this MR do and why?

For #329330 (closed)

This MR duplicates the "Assignees" behavior for un-collapsed assignees by:

  • Adding the user's defined name, and
  • Increasing the size of the user's avatar to match the extra height

Notable exclusions

Please note that this MR explicitly does not include the Reviewers' Busy status.

While it is likely useful to see that information, the Reviewer-type user provided to the Reviewers list lacks the availability or status fields that are included with the Assignee-type users provided to the Assignees list.

Screenshots or screen recordings

Before After
before after

How to set up and validate locally

  1. Create an MR
  2. Assign a reviewer to it
  3. Gaze in awe upon the reviewer's name
  4. (optional) log in as the reviewer
  5. (optional) approve the MR as the reviewer
  6. (not optional) gaze in awe upon the green circle with the white check mark in it.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Thomas Randolph

Merge request reports