Merge pull request #722 from REBELinBLUE/slack-bug-fix
Fixing issue with status overwriting message
This commit is contained in:
commit
e9e0ba8593
|
@ -81,10 +81,10 @@ class SlackNotify implements \PHPCI\Plugin
|
||||||
$successfulBuild = $this->build->isSuccessful();
|
$successfulBuild = $this->build->isSuccessful();
|
||||||
|
|
||||||
if ($successfulBuild) {
|
if ($successfulBuild) {
|
||||||
$message = 'Success';
|
$status = 'Success';
|
||||||
$color = 'good';
|
$color = 'good';
|
||||||
} else {
|
} else {
|
||||||
$message = 'Failed';
|
$status = 'Failed';
|
||||||
$color = 'danger';
|
$color = 'danger';
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -96,7 +96,7 @@ class SlackNotify implements \PHPCI\Plugin
|
||||||
'fields' => array(
|
'fields' => array(
|
||||||
new \Maknz\Slack\AttachmentField(array(
|
new \Maknz\Slack\AttachmentField(array(
|
||||||
'title' => 'Status',
|
'title' => 'Status',
|
||||||
'value' => $message,
|
'value' => $status,
|
||||||
'short' => false
|
'short' => false
|
||||||
))
|
))
|
||||||
)
|
)
|
||||||
|
|
Loading…
Reference in a new issue