Skip to content

Commit 42a91b4

Browse files
author
Desjardins Jérôme
committed
Channel not required for webhook, can be configurated whith webhook
1 parent ec5c789 commit 42a91b4

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

DependencyInjection/Configuration.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -369,7 +369,7 @@ public function getConfigTreeBuilder()
369369
->scalarNode('room')->end() // hipchat
370370
->scalarNode('message_format')->defaultValue('text')->end() // hipchat
371371
->scalarNode('api_version')->defaultNull()->end() // hipchat
372-
->scalarNode('channel')->end() // slack & slackwebhook & slackbot
372+
->scalarNode('channel')->defaultNull()->end() // slack & slackwebhook & slackbot
373373
->scalarNode('bot_name')->defaultValue('Monolog')->end() // slack & slackwebhook
374374
->scalarNode('use_attachment')->defaultTrue()->end() // slack & slackwebhook
375375
->scalarNode('use_short_attachment')->defaultFalse()->end() // slack & slackwebhook
@@ -713,8 +713,8 @@ public function getConfigTreeBuilder()
713713
->thenInvalid('The token and channel have to be specified to use a SlackHandler')
714714
->end()
715715
->validate()
716-
->ifTrue(function ($v) { return 'slackwebhook' === $v['type'] && (empty($v['webhook_url']) || empty($v['channel'])); })
717-
->thenInvalid('The webhook_url and channel have to be specified to use a SlackWebhookHandler')
716+
->ifTrue(function ($v) { return 'slackwebhook' === $v['type'] && (empty($v['webhook_url'])); })
717+
->thenInvalid('The webhook_url have to be specified to use a SlackWebhookHandler')
718718
->end()
719719
->validate()
720720
->ifTrue(function ($v) { return 'slackbot' === $v['type'] && (empty($v['stack_team']) || empty($v['token']) || empty($v['channel'])); })

0 commit comments

Comments
 (0)