Return early if user is already approved the merge request
requested to merge batu/gitlab:372972-finish-review-fails-when-approve-box-is-checked-on-already-approved-mr into master
What does this MR do and why?
- Added
already_approved_by?(user)method to Merge Request model to check if given user has any approvals. - In drafts controller, returns early if merge request is already approved by the user.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #372972 (closed)
Edited by Batuhan Wilhelm