summaryrefslogtreecommitdiff
path: root/app/services/members
Commit message (Collapse)AuthorAgeFilesLines
* Implement CreateMembers service to make controller thinrefactor_group_membersValery Sizov2016-11-011-0/+16
|
* Allow Members::ApproveAccessRequestService to accept a new :force paramee-1159-allow-permission-check-bypass-in-approve-access-request-serviceRémy Coutable2016-10-281-5/+16
| | | | | | | | | | | This param allows to bypass permission check. It is useful for LDAP-sync where even owners don't have the :admin_group_member permission. See https://gitlab.com/gitlab-org/gitlab-ee/blob/6081c37123abae4570f78831b33c2f45f92c2765/app/policies/group_policy.rb#L38 and https://gitlab.com/gitlab-org/gitlab-ee/issues/1159 Signed-off-by: Rémy Coutable <remy@rymai.me>
* Invert method's namingRémy Coutable2016-10-031-3/+3
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Fix a few things after the initial improvment to Members::DestroyServiceRémy Coutable2016-10-032-5/+8
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Improve Members::DestroyServiceRémy Coutable2016-10-031-7/+31
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Merge branch 'rc-new-members-request-access-service' into 'master' Robert Speicher2016-10-021-0/+25
|\ | | | | | | | | | | | | New `Members::RequestAccessService` Part of #21979. See merge request !6265
| * Inverse condition in Members::RequestAccessServiceRémy Coutable2016-09-221-3/+3
| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
| * New Members::RequestAccessServiceRémy Coutable2016-09-221-0/+25
| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Inverse condition in Members::ApproveAccessRequestServicerc-new-members-approve-request-access-serviceRémy Coutable2016-09-221-3/+3
| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Ensure Members::ApproveAccessRequestService can fin a requester by IDRémy Coutable2016-09-221-1/+2
| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | New Members::ApproveAccessRequestServiceRémy Coutable2016-09-221-0/+30
|/ | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Ensure last group owner isn't removed on expirySean McGivern2016-08-181-0/+2
|
* Merge branch 'master' into expiration-date-on-membershipsSean McGivern2016-08-181-2/+3
|\
| * New AccessRequests API endpoints for Group & ProjectRémy Coutable2016-08-101-2/+3
| | | | | | | | | | | | | | | | Also, mutualize AccessRequests and Members endpoints for Group & Project. New API documentation for the AccessRequests endpoints. Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Extract Members::AuthorizedDestroyService from Members::DestroyService.Adam Niedzielski2016-08-042-6/+18
|/
* Raise a new Gitlab::Access::AccessDeniedError when permission is not enough ↵Rémy Coutable2016-06-181-20/+6
| | | | | | | | to destroy a member This is a try for a new approach to put the access checks at the service level. Signed-off-by: Rémy Coutable <remy@rymai.me>
* Don't send the "access declined" email on access request withdrawalRémy Coutable2016-06-181-1/+3
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* New Members::DestroyServiceRémy Coutable2016-06-181-0/+33
This is to ensure we don't send unwanted notifications when deleting a project. In other words, stop abusing AR callbacks and use services. Signed-off-by: Rémy Coutable <remy@rymai.me>