make sure users have to be able to read_pipeline

to get pipeline failed notifications

make sure users have to be able to read_pipeline
to get pipeline failed notifications
d45b4065 · http://jneen.net/ · 46d0bfdc · d45b4065
Commit d45b4065 authored Jul 26, 2017 by http://jneen.net/
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

app/services/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
@@ -308,7 +308,8 @@ class NotificationService

    recipients ||= NotificationRecipientService.notifiable_users(
      [pipeline.user], pipeline.project, :watch,
-      custom_action: :"#{pipeline.status}_pipeline"
+      custom_action: :"#{pipeline.status}_pipeline",
+      read_ability: :read_pipeline,
    ).map(&:notification_email)

    if recipients.any?