delta/gitlab/gitlab-ce.git/lib/api/members.rb, branch gitlab-git-duplicate-code gitlab.com: gitlab-org/gitlab-ce.git Improve Member services 2018-02-27T15:02:22+00:00 Rémy Coutable remy@rymai.me 2018-02-16T14:10:22+00:00 1c88d92b3fe174a56080575a14d6b473f17f7d8f Signed-off-by: Rémy Coutable <remy@rymai.me> 
Signed-off-by: Rémy Coutable <remy@rymai.me>
Remove explicit audit event log in MembershipActions 2018-02-27T15:02:22+00:00 Rémy Coutable remy@rymai.me 2017-10-11T14:47:08+00:00 bf41063679b25371b2e64542f2f469b38502edf6 Move it to Members::ApproveAccessRequestService. Also, note that there was a double audit event log for access request destruction. Signed-off-by: Rémy Coutable <remy@rymai.me> 
Move it to Members::ApproveAccessRequestService.

Also, note that there was a double audit event log for access request
destruction.

Signed-off-by: Rémy Coutable <remy@rymai.me>
Remove N+1 queries with /projects/:project_id/{access_requests,members} API endpoints 2018-01-31T05:46:09+00:00 Stan Hu stanhu@gmail.com 2018-01-27T06:16:56+00:00 46a6edc7314ce8acab5d8ce04799bd3557bc26bc We can simplify the code quite a bit and improve performance by using grape-entity merge fields: https://github.com/ruby-grape/grape-entity/tree/v0.6.0#merge-fields Relates to #42030 
We can simplify the code quite a bit and improve performance by using
grape-entity merge fields:

https://github.com/ruby-grape/grape-entity/tree/v0.6.0#merge-fields

Relates to #42030
Fix error on empty query for Members API 2018-01-21T16:36:55+00:00 Mark Fletcher mark@gitlab.com 2018-01-04T16:35:44+00:00 062f5b7126978e08522361e879d7c1723b15c9da 






Backport a change made in EE
2017-12-21T10:15:19+00:00

Rémy Coutable
remy@rymai.me

2017-12-20T16:08:28+00:00

51b416338a2ee9e287787850d11a3474e16f1474

Signed-off-by: Rémy Coutable <remy@rymai.me>





Signed-off-by: Rémy Coutable <remy@rymai.me>







API: Use defined project requirements
2017-08-31T12:57:47+00:00

Robert Schilling
rschilling@student.tugraz.at

2017-08-31T11:44:49+00:00

6a2ee0968e811d31fb4cc23b30a6b42e42adf47b












Conditionally destroy a ressource
2017-08-28T14:40:25+00:00

Robert Schilling
rschilling@student.tugraz.at

2017-03-02T12:14:13+00:00

e80313f9ee5b3495a8713e6ddae111bc8106155b












API: Respect the 'If-Unmodified-Since' for delete endpoints
2017-08-28T14:40:25+00:00

Robert Schilling
rschilling@student.tugraz.at

2017-03-01T13:35:48+00:00

998afa5f74558be215a924d95aa131a69831ca43












Update grape gem
2017-07-20T13:33:18+00:00

Dmitriy Zaporozhets
dmitriy.zaporozhets@gmail.com

2017-07-20T13:33:18+00:00

6b8ad689da393125bb2d1e548211c9a50039b0a7

New version of the gem returns 200 status code on delete with content
instead of 204 so we explicitly set status code to keep existing
behavior

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>





New version of the gem returns 200 status code on delete with content
instead of 204 so we explicitly set status code to keep existing
behavior

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>







Add `requirements: { id: %r{[^/]+} }` for all projects and groups namespaced API routes
2017-03-16T17:00:24+00:00

Rémy Coutable
remy@rymai.me

2017-03-15T18:09:24+00:00

63360adeae3ba5b302b711c73b6439956d274cde

Signed-off-by: Rémy Coutable <remy@rymai.me>





Signed-off-by: Rémy Coutable <remy@rymai.me>