- Notifications
You must be signed in to change notification settings - Fork673
Commit88ac96d
committed
fix: members: use new *All objects for *AllManager managers
Change it so that: GroupMemberAllManager uses GroupMemberAll object ProjectMemberAllManager uses ProjectMemberAll objectCreate GroupMemberAll and ProjectMemberAll objects that do not supportany Mixin type methods. Previously we were using GroupMember andProjectMember which support the `save()` and `delete()` methods butthose methods will not work with objects retrieved using the`/members/all/` API calls.`list()` API calls: [1] GET /groups/:id/members/all GET /projects/:id/members/all`get()` API calls: [2] GET /groups/:id/members/all/:user_id GET /projects/:id/members/all/:user_idCloses:#1825[1]https://docs.gitlab.com/ee/api/members.html#list-all-members-of-a-group-or-project-including-inherited-and-invited-members[2]https://docs.gitlab.com/ee/api/members.html#get-a-member-of-a-group-or-project-including-inherited-and-invited-members1 parent4a000b6 commit88ac96d
1 file changed
+14
-6
lines changedLines changed: 14 additions & 6 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
70 | 70 |
| |
71 | 71 |
| |
72 | 72 |
| |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
73 | 77 |
| |
74 | 78 |
| |
75 |
| - | |
| 79 | + | |
76 | 80 |
| |
77 | 81 |
| |
78 | 82 |
| |
79 | 83 |
| |
80 |
| - | |
81 |
| - | |
| 84 | + | |
| 85 | + | |
82 | 86 |
| |
83 | 87 |
| |
84 | 88 |
| |
| |||
103 | 107 |
| |
104 | 108 |
| |
105 | 109 |
| |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
106 | 114 |
| |
107 | 115 |
| |
108 |
| - | |
| 116 | + | |
109 | 117 |
| |
110 | 118 |
| |
111 | 119 |
| |
112 | 120 |
| |
113 |
| - | |
114 |
| - | |
| 121 | + | |
| 122 | + |
0 commit comments
Comments
(0)