| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
1 parent 7843ace commit e512cdd
1 file changed
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -2685,10 +2685,11 @@ class ProjectMergeRequestApprovalManager(GetWithoutIdMixin, UpdateMixin, RESTMan | |||
| 2685 | 2685 | _update_uses_post = True | |
| 2686 | 2686 | ||
| 2687 | 2687 | @exc.on_http_error(exc.GitlabUpdateError) | |
| 2688 | - def set_approvers(self, approver_ids=None, approver_group_ids=None, **kwargs): | ||
| 2688 | + def set_approvers(self, approvals_required, approver_ids=None, approver_group_ids=None, **kwargs): | ||
| 2689 | 2689 | """Change MR-level allowed approvers and approver groups. | |
| 2690 | 2690 | ||
| 2691 | 2691 | Args: | |
| 2692 | + approvals_required (integer): The number of required approvals for this rule | ||
| 2692 | 2693 | approver_ids (list): User IDs that can approve MRs | |
| 2693 | 2694 | approver_group_ids (list): Group IDs whose members can approve MRs | |
| 2694 | 2695 | ||
@@ -2699,8 +2700,12 @@ def set_approvers(self, approver_ids=None, approver_group_ids=None, **kwargs): | |||
| 2699 | 2700 | approver_ids = approver_ids or [] | |
| 2700 | 2701 | approver_group_ids = approver_group_ids or [] | |
| 2701 | 2702 | ||
| 2702 | - path = "%s/%s/approvers" % (self._parent.manager.path, self._parent.get_id()) | ||
| 2703 | - data = {"approver_ids": approver_ids, "approver_group_ids": approver_group_ids} | ||
| 2703 | + path = "%s/%s/approval_rules" % (self._parent.manager.path, self._parent.get_id()) | ||
| 2704 | + data = { | ||
| 2705 | + "approvals_required": approvals_required, | ||
| 2706 | + "rule_type": "regular", | ||
| 2707 | + "user_ids": approver_ids, | ||
| 2708 | + "group_ids": approver_group_ids} | ||
| 2704 | 2709 | self.gitlab.http_put(path, post_data=data, **kwargs) | |
| 2705 | 2710 | ||
| 2706 | 2711 | ||
| Back | FazBrowse Home | New Git URL |
0 commit comments