Skip to content
Snippets Groups Projects
Commit 1cdad622 authored by Lin Jen-Shin's avatar Lin Jen-Shin
Browse files

Should send to notification_email instead, feedback:

parent a87b6cc3
No related branches found
No related tags found
1 merge request!6342Integrate CI emails into notification system
Pipeline #
Loading
@@ -321,7 +321,7 @@ class NotificationService
Loading
@@ -321,7 +321,7 @@ class NotificationService
pipeline, pipeline,
pipeline.project, pipeline.project,
nil, # The acting user, who won't be added to recipients nil, # The acting user, who won't be added to recipients
action: pipeline.status).map(&:email) action: pipeline.status).map(&:notification_email)
   
mailer.public_send(email_template, pipeline, recipients).deliver_later mailer.public_send(email_template, pipeline, recipients).deliver_later
end end
Loading
Loading
Loading
@@ -58,7 +58,7 @@ describe PipelinesEmailService do
Loading
@@ -58,7 +58,7 @@ describe PipelinesEmailService do
end end
   
it 'sends email' do it 'sends email' do
should_only_email(double(email: recipient), kind: :bcc) should_only_email(double(notification_email: recipient), kind: :bcc)
end end
end end
   
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment