Skip to content

Approve additionally

Jose Ivan Vargas requested to merge 4134-approve-additionally into master

What does this MR do?

Changes the current behavior for approving merge requests by allowing them to be approved with more people even if the merge request itself already has enough approvals.

Screenshots

3 approvals (approved additionally)

Screen_Shot_2018-01-30_at_11.14.06_AM

The approve additionally button

Screen_Shot_2018-01-30_at_11.14.16_AM

Requires n approvals

Screen_Shot_2018-02-01_at_11.57.55_AM

Requires n more approvals with the remove approval button

Screen_Shot_2018-02-01_at_12.01.01_PM

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #4134 (closed)

Edited by Tim Zallmann

Merge request reports