Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Implement CreateMembers service to make controller thinrefactor_group_members | Valery Sizov | 2016-11-01 | 1 | -0/+16 |
| | |||||
* | Allow Members::ApproveAccessRequestService to accept a new :force paramee-1159-allow-permission-check-bypass-in-approve-access-request-service | Rémy Coutable | 2016-10-28 | 1 | -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 naming | Rémy Coutable | 2016-10-03 | 1 | -3/+3 |
| | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Fix a few things after the initial improvment to Members::DestroyService | Rémy Coutable | 2016-10-03 | 2 | -5/+8 |
| | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Improve Members::DestroyService | Rémy Coutable | 2016-10-03 | 1 | -7/+31 |
| | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Merge branch 'rc-new-members-request-access-service' into 'master' | Robert Speicher | 2016-10-02 | 1 | -0/+25 |
|\ | | | | | | | | | | | | | New `Members::RequestAccessService` Part of #21979. See merge request !6265 | ||||
| * | Inverse condition in Members::RequestAccessService | Rémy Coutable | 2016-09-22 | 1 | -3/+3 |
| | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
| * | New Members::RequestAccessService | Rémy Coutable | 2016-09-22 | 1 | -0/+25 |
| | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | | Inverse condition in Members::ApproveAccessRequestServicerc-new-members-approve-request-access-service | Rémy Coutable | 2016-09-22 | 1 | -3/+3 |
| | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | | Ensure Members::ApproveAccessRequestService can fin a requester by ID | Rémy Coutable | 2016-09-22 | 1 | -1/+2 |
| | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | | New Members::ApproveAccessRequestService | Rémy Coutable | 2016-09-22 | 1 | -0/+30 |
|/ | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | Ensure last group owner isn't removed on expiry | Sean McGivern | 2016-08-18 | 1 | -0/+2 |
| | |||||
* | Merge branch 'master' into expiration-date-on-memberships | Sean McGivern | 2016-08-18 | 1 | -2/+3 |
|\ | |||||
| * | New AccessRequests API endpoints for Group & Project | Rémy Coutable | 2016-08-10 | 1 | -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 Niedzielski | 2016-08-04 | 2 | -6/+18 |
|/ | |||||
* | Raise a new Gitlab::Access::AccessDeniedError when permission is not enough ↵ | Rémy Coutable | 2016-06-18 | 1 | -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 withdrawal | Rémy Coutable | 2016-06-18 | 1 | -1/+3 |
| | | | | Signed-off-by: Rémy Coutable <remy@rymai.me> | ||||
* | New Members::DestroyService | Rémy Coutable | 2016-06-18 | 1 | -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> |