Skip to content

Commit a216955

Browse files
authored
bug: fix notifications when not using thresholds (alexjustesen#2464)
1 parent b9170ce commit a216955

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

app/Listeners/ProcessCompletedSpeedtest.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ private function notifyAppriseChannels(Result $result): void
5555
private function notifyDatabaseChannels(Result $result): void
5656
{
5757
// Don't send database notification if dispatched by a user or test is unhealthy.
58-
if (filled($result->dispatched_by) || ! $result->healthy) {
58+
if (filled($result->dispatched_by) || ! $result->healthy === false) {
5959
return;
6060
}
6161

@@ -104,7 +104,7 @@ private function notifyDispatchingUser(Result $result): void
104104
*/
105105
private function notifyMailChannels(Result $result): void
106106
{
107-
if (filled($result->dispatched_by) || ! $result->healthy) {
107+
if (filled($result->dispatched_by) || $result->healthy === false) {
108108
return;
109109
}
110110

@@ -130,7 +130,7 @@ private function notifyMailChannels(Result $result): void
130130
private function notifyWebhookChannels(Result $result): void
131131
{
132132
// Don't send webhook if dispatched by a user or test is unhealthy.
133-
if (filled($result->dispatched_by) || ! $result->healthy) {
133+
if (filled($result->dispatched_by) || $result->healthy === false) {
134134
return;
135135
}
136136

0 commit comments

Comments
 (0)