From 4b7f035aa86bde3b14f761b52e1634d8c13f59c2 Mon Sep 17 00:00:00 2001 From: Sean McGivern Date: Thu, 5 Jul 2018 12:58:21 +0100 Subject: [PATCH] Extract EE-specific lines from NotificationService --- app/services/notification_service.rb | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/app/services/notification_service.rb b/app/services/notification_service.rb index 636cfbf5b45..8c6221af788 100644 --- a/app/services/notification_service.rb +++ b/app/services/notification_service.rb @@ -135,6 +135,8 @@ class NotificationService # * watchers of the mr's labels # * users with custom level checked with "new merge request" # + # In EE, approvers of the merge request are also included + # def new_merge_request(merge_request, current_user) new_resource_email(merge_request, :new_merge_request_email) end @@ -256,6 +258,10 @@ class NotificationService # ignore gitlab service messages return true if note.cross_reference? && note.system? + send_new_note_notifications(note) + end + + def send_new_note_notifications(note) notify_method = "note_#{note.to_ability_name}_email".to_sym recipients = NotificationRecipientService.build_new_note_recipients(note)