It no longer needs a block, feedback:

https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/6342#note_15434727
This commit is contained in:
Lin Jen-Shin 2016-09-19 13:44:36 +08:00
parent 55f2e6ab21
commit bac500fdeb

View file

@ -1,9 +1,7 @@
class PipelinesEmailService < Service
prop_accessor :recipients
boolean_accessor :notify_only_broken_pipelines
validates :recipients,
presence: true,
if: ->(s) { s.activated? }
validates :recipients, presence: true, if: :activated?
def initialize_properties
self.properties ||= { notify_only_broken_pipelines: true }