From c8558b5f7885b91711429b3c16a78089de81af97 Mon Sep 17 00:00:00 2001
From: Andras Bacsai <5845193+andrasbacsai@users.noreply.github.com>
Date: Tue, 18 Nov 2025 15:13:55 +0100
Subject: [PATCH] fix: add Update button for locked environment variable
 comments
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Removed instantSave from comment field and added a proper Update button with Delete modal for locked environment variables. This ensures users can explicitly save their comment changes on locked variables.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 .../shared/environment-variable/show.blade.php      | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/resources/views/livewire/project/shared/environment-variable/show.blade.php b/resources/views/livewire/project/shared/environment-variable/show.blade.php
index 43eb88335..b41d223ce 100644
--- a/resources/views/livewire/project/shared/environment-variable/show.blade.php
+++ b/resources/views/livewire/project/shared/environment-variable/show.blade.php
@@ -23,7 +23,7 @@
             </div>
             @can('update', $this->env)
                 <div class="w-full">
-                    <x-forms.input instantSave id="comment" label="Comment (Optional)"
+                    <x-forms.input id="comment" label="Comment (Optional)"
                         helper="Add a note to document what this environment variable is used for." maxlength="256" />
                 </div>
                 <div class="flex flex-col w-full gap-3">
@@ -71,6 +71,17 @@
                         @endif
                     </div>
                 </div>
+                <div class="flex w-full justify-end gap-2">
+                    <x-forms.button type="submit">Update</x-forms.button>
+                    @can('delete', $this->env)
+                        <x-modal-confirmation title="Confirm Environment Variable Deletion?" isErrorButton
+                            buttonTitle="Delete" submitAction="delete" :actions="['The selected environment variable will be permanently deleted.']"
+                            confirmationText="{{ $env->key }}" buttonFullWidth="true"
+                            confirmationLabel="Please confirm the execution of the actions by entering the Environment Variable Name below"
+                            shortConfirmationLabel="Environment Variable Name" :confirmWithPassword="false"
+                            step2ButtonText="Permanently Delete" />
+                    @endcan
+                </div>
             @else
                 <div class="flex flex-col w-full gap-3">
                     <div class="flex flex-wrap w-full items-center gap-4">