Skip to content

Fixes a bug with assigning a reviewer and requested changes

Phil Hughes requested to merge ph/480177/addRequestedChangesToNewReviewers into master

What does this MR do and why?

Fixes a bug where assigning a reviewer who has already requested changes would not have their state set correctly.

#480177 (closed)

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

How to set up and validate locally

  • On a merge request that you are not a reviewer of submit a new review and request changes
  • Assign yourself as a reviewer
  • The requested changes icon should be visible in the sidebar

Merge request reports

Loading