From e8f3aa681e1305ff7d90b2638359a5725dc731bb Mon Sep 17 00:00:00 2001 From: Andras Bacsai Date: Wed, 13 Mar 2024 10:24:45 +0100 Subject: [PATCH] fix: failed deployments should send failed email/notification --- app/Jobs/ApplicationDeploymentJob.php | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/app/Jobs/ApplicationDeploymentJob.php b/app/Jobs/ApplicationDeploymentJob.php index b9c60605f..fb94a6694 100644 --- a/app/Jobs/ApplicationDeploymentJob.php +++ b/app/Jobs/ApplicationDeploymentJob.php @@ -1678,12 +1678,14 @@ COPY ./nginx.conf /etc/nginx/conf.d/default.conf"); { queue_next_deployment($this->application); // If the deployment is cancelled by the user, don't update the status - if ($this->application_deployment_queue->status !== ApplicationDeploymentStatus::CANCELLED_BY_USER->value && $this->application_deployment_queue->status !== ApplicationDeploymentStatus::FAILED->value) { + if ( + $this->application_deployment_queue->status !== ApplicationDeploymentStatus::CANCELLED_BY_USER->value && $this->application_deployment_queue->status !== ApplicationDeploymentStatus::FAILED->value + ) { $this->application_deployment_queue->update([ 'status' => $status, ]); } - if ($status === ApplicationDeploymentStatus::FAILED->value) { + if ($this->application_deployment_queue->status === ApplicationDeploymentStatus::FAILED->value) { $this->application->environment->project->team?->notify(new DeploymentFailed($this->application, $this->deployment_uuid, $this->preview)); return; }