Skip to content
Snippets Groups Projects

New `Members::ApproveAccessRequestService`

1 unresolved thread

Part of #21979 (closed).

Does this MR meet the acceptance criteria?

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
2 class ApproveAccessRequestService < BaseService
3 include MembersHelper
4
5 attr_accessor :source
6
7 def initialize(source, current_user, params = {})
8 @source = source
9 @current_user = current_user
10 @params = params
11 end
12
13 def execute
14 condition = params[:user_id] ? { user_id: params[:user_id] } : { id: params[:id] }
15 access_requester = source.requesters.find_by!(condition)
16
17 raise Gitlab::Access::AccessDeniedError if cannot_update_access_requester?(access_requester)
  • Added 596 commits:

    • 65581360...8071dc83 - 593 commits from branch master
    • b3f0a82f - New Members::ApproveAccessRequestService
    • 5dcdf1d5 - Ensure Members::ApproveAccessRequestService can fin a requester by ID
    • 94996963 - Inverse condition in Members::ApproveAccessRequestService
  • Douwe Maan Status changed to merged

    Status changed to merged

  • Douwe Maan Mentioned in commit 166c6cd8

    Mentioned in commit 166c6cd8

  • Mentioned in merge request !6269 (merged)

  • Robert Speicher Mentioned in commit 8a96910c

    Mentioned in commit 8a96910c

  • Please register or sign in to reply
    Loading