require that the user be able to :read_build

to get a pipeline_failed email

require that the user be able to :read_build
3829d724 · http://jneen.net/ · d45b4065 · 3829d724
Commit 3829d724 authored Jul 28, 2017 by http://jneen.net/
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

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

No files found.
--- a/app/services/notification_service.rb
+++ b/app/services/notification_service.rb
@@ -309,7 +309,8 @@ class NotificationService
    recipients ||= NotificationRecipientService.notifiable_users(
      [pipeline.user], pipeline.project, :watch,
      custom_action: :"#{pipeline.status}_pipeline",
-      read_ability: :read_pipeline,
+      read_ability: :read_build,
+      target: pipeline
    ).map(&:notification_email)

    if recipients.any?