Skip to content
Snippets Groups Projects

Fix Approval UI showing up for free plan

Merged Mark Chao requested to merge 9908-hide-approver-ui-in-free into master
All threads resolved!
3 files
+ 8
2
Compare changes
  • Side-by-side
  • Inline
Files
3
  • b56c5466
    Fix MR approval form showing in free plan · b56c5466
    Mark Chao authored
    `requires_approve?` went through several iterations,
    earliest one implies license by checking `approvals_required.nonzero?`.
    This was however set to true so caused EE specific UI to appear by
    mistake.
@@ -3,7 +3,7 @@
- form = local_assigns.fetch(:form)
- return unless issuable.is_a?(MergeRequest)
- return unless presenter.requires_approve?
- return unless @project.feature_available?(:merge_request_approvers)
- can_update_approvers = can?(current_user, :update_approvers, issuable)
Loading