From 89a66dd255262f9a105dc9d4007c8aefe9f8456a Mon Sep 17 00:00:00 2001
From: Andras Bacsai <5845193+andrasbacsai@users.noreply.github.com>
Date: Fri, 4 Sep 2026 06:30:35 +0200
Subject: [PATCH] fix(service): show disabled deploy action when variables are
 missing

---
 .../views/livewire/project/service/heading.blade.php   |  6 ++++++
 tests/Feature/ResourceHeadingUnifiedNavbarTest.php     | 10 +++++++++-
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/resources/views/livewire/project/service/heading.blade.php b/resources/views/livewire/project/service/heading.blade.php
index e7da50c80..79899e879 100644
--- a/resources/views/livewire/project/service/heading.blade.php
+++ b/resources/views/livewire/project/service/heading.blade.php
@@ -188,6 +188,12 @@ class="listbox-panel top-full! left-0! right-0! mt-1! w-full! min-w-0!" role="me
                 </div>
                 @endcan
             @else
+                @can('deploy', $service)
+                    <button type="button" class="button mb-2 w-full justify-between" disabled>
+                        <span>Deploy</span>
+                        <span class="text-xs font-normal opacity-70">Fill required variables first</span>
+                    </button>
+                @endcan
                 <a href="{{ $environmentVariablesUrl }}" {{ wireNavigate() }}
                     class="mb-3 inline-flex" aria-label="Open required environment variables">
                     <x-status-badge status="Required variables missing" type="error" />
diff --git a/tests/Feature/ResourceHeadingUnifiedNavbarTest.php b/tests/Feature/ResourceHeadingUnifiedNavbarTest.php
index 330afa7ed..ce69f26ef 100644
--- a/tests/Feature/ResourceHeadingUnifiedNavbarTest.php
+++ b/tests/Feature/ResourceHeadingUnifiedNavbarTest.php
@@ -84,11 +84,19 @@
 
 it('links the service header missing variables warning to environment variables', function () {
     $heading = file_get_contents(resource_path('views/livewire/project/service/heading.blade.php'));
+    $mobileActions = str($heading)
+        ->after('<div class="w-full xl:hidden">')
+        ->before("@teleport('#resource-action-hud-slot')")
+        ->toString();
 
     expect($heading)
         ->toContain("route('project.service.environment-variables'")
         ->toContain('Required variables missing')
-        ->toContain('href="{{ $environmentVariablesUrl }}"');
+        ->toContain('href="{{ $environmentVariablesUrl }}"')
+        ->and($mobileActions)
+        ->toContain('Fill required variables first')
+        ->toContain('disabled')
+        ->toContain('Deploy');
 });
 
 it('places the account menu beside the desktop sidebar toggle while retaining it on mobile', function () {