diff options
author | http://jneen.net/ <jneen@jneen.net> | 2017-07-26 19:33:23 +0300 |
---|---|---|
committer | http://jneen.net/ <jneen@jneen.net> | 2017-08-03 19:07:18 +0300 |
commit | d45b4065d8e0b6cc893ccde66e0c37f459ca901b (patch) | |
tree | 99d26b849faa2863af0f07dc5ef539927268f4bd /app/services/notification_service.rb | |
parent | 46d0bfdcb982d76b8595921a956b088df1573905 (diff) |
make sure users have to be able to read_pipeline
to get pipeline failed notifications
Diffstat (limited to 'app/services/notification_service.rb')
-rw-r--r-- | app/services/notification_service.rb | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/app/services/notification_service.rb b/app/services/notification_service.rb index 675b4536a26..c330d62a2ce 100644 --- 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? |