recipients should be a list of emails

691ed8ac · Lin Jen-Shin · 1c38dbca · 691ed8ac
Commit 691ed8ac authored Sep 14, 2016 by Lin Jen-Shin
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

app/services/notification_service.rb app/services/notification_service.rb +2 -2

No files found.
--- a/app/services/notification_service.rb
+++ b/app/services/notification_service.rb
@@ -321,10 +321,10 @@ class NotificationService
      pipeline,
      pipeline.project,
      nil, # The acting user, who won't be added to recipients
-      action: pipeline.status)
+      action: pipeline.status).map(&:email)

    recipients.each do |to|
-      mailer.public_send(email_template, pipeline, to.email).deliver_later
+      mailer.public_send(email_template, pipeline, to).deliver_later
    end
  end