gitlab-org--gitlab-foss/app/services/merge_requests/resolved_discussion_notific...

# frozen_string_literal: true

module MergeRequests
  class ResolvedDiscussionNotificationService < MergeRequests::BaseService
    def execute(merge_request)
      return unless merge_request.discussions_resolved?

      SystemNoteService.resolve_all_discussions(merge_request, project, current_user)
      execute_hooks(merge_request, 'update')
      notification_service.async.resolve_all_discussions(merge_request, current_user)
    end
  end
end
Enable more frozen string in app/services/*/.rb Partially addresses #47424. 2018-07-17 16:50:37 +00:00			`# frozen_string_literal: true`

Add 'Resolved all discussions' system note 2016-07-29 02:39:35 +00:00			`module MergeRequests`
Address review feedback 2016-08-15 23:45:23 +00:00			`class ResolvedDiscussionNotificationService < MergeRequests::BaseService`
Add 'Resolved all discussions' system note 2016-07-29 02:39:35 +00:00			`def execute(merge_request)`
			`return unless merge_request.discussions_resolved?`

			`SystemNoteService.resolve_all_discussions(merge_request, project, current_user)`
Add latest changes from gitlab-org/gitlab@master 2021-11-19 12:12:41 +00:00			`execute_hooks(merge_request, 'update')`
Move NotificationService calls to Sidekiq The NotificationService has to do quite a lot of work to calculate the recipients for an email. Where possible, we should try to avoid doing this in an HTTP request, because the mail are sent by Sidekiq anyway, so there's no need to schedule those emails immediately. This commit creates a generic Sidekiq worker that uses Global ID to serialise and deserialise its arguments, then forwards them to the NotificationService. The NotificationService gains an `#async` method, so you can replace: notification_service.new_issue(issue, current_user) With: notification_service.async.new_issue(issue, current_user) And have everything else work as normal, except that calculating the recipients will be done by Sidekiq, which will then schedule further Sidekiq jobs to send each email. 2018-04-20 17:37:38 +00:00			`notification_service.async.resolve_all_discussions(merge_request, current_user)`
Add 'Resolved all discussions' system note 2016-07-29 02:39:35 +00:00			`end`
			`end`
			`end`