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!
Compare changes
  • Side-by-side
  • Inline
Files
10
  • - Removes `requires_approve?`. This was only being used
       in `merge_request_presenter` to hide/show the API paths.
    - Adds `has_approvals_available?` which checks if the feature
       is available. The API path checks have been updated to use
       this call.
    - Replaces `approvalsRequired` in the mr_widget_store with
       `hasApprovalsAvailable`. The previous property actually
       only checked if the `approvalsPath` was truthy or not. It
       looked like it checked the `approvalsRequired` from the
       server, but since this was camel case, this value never existed.
Loading