From 9d5675ed62fd1a058440383b0a2c1dfeefa2de25 Mon Sep 17 00:00:00 2001
From: Torsten Dittmann <torsten.dittmann@googlemail.com>
Date: Mon, 19 May 2025 00:41:18 +0200
Subject: [PATCH] Allow updating of secret variables

---
 .../updateVariables.svelte                    | 19 +++++++++----------
 .../updateVariablesModal.svelte               |  1 +
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/routes/(console)/project-[region]-[project]/updateVariables.svelte b/src/routes/(console)/project-[region]-[project]/updateVariables.svelte
index 2bc5f2707..2247a4dae 100644
--- a/src/routes/(console)/project-[region]-[project]/updateVariables.svelte
+++ b/src/routes/(console)/project-[region]-[project]/updateVariables.svelte
@@ -69,7 +69,6 @@
 
     async function handleVariableCreated(event: CustomEvent<Models.Variable[]>) {
         const variables = event.detail;
-        console.log(variables);
         try {
             const promises = variables.map((variable) =>
                 sdkCreateVariable(variable.key, variable.value, variable?.secret || false)
@@ -369,16 +368,16 @@
                                     </Button>
                                     <svelte:fragment slot="tooltip" let:toggle>
                                         <ActionMenu.Root>
+                                            <ActionMenu.Item.Button
+                                                trailingIcon={IconPencil}
+                                                on:click={(e) => {
+                                                    selectedVar = variable;
+                                                    showUpdate = true;
+                                                    toggle(e);
+                                                }}>
+                                                Update
+                                            </ActionMenu.Item.Button>
                                             {#if !variable.secret}
-                                                <ActionMenu.Item.Button
-                                                    trailingIcon={IconPencil}
-                                                    on:click={(e) => {
-                                                        selectedVar = variable;
-                                                        showUpdate = true;
-                                                        toggle(e);
-                                                    }}>
-                                                    Update
-                                                </ActionMenu.Item.Button>
                                                 <ActionMenu.Item.Button
                                                     trailingIcon={IconEyeOff}
                                                     on:click={(e) => {
diff --git a/src/routes/(console)/project-[region]-[project]/updateVariablesModal.svelte b/src/routes/(console)/project-[region]-[project]/updateVariablesModal.svelte
index 88e2387b6..58e6026c0 100644
--- a/src/routes/(console)/project-[region]-[project]/updateVariablesModal.svelte
+++ b/src/routes/(console)/project-[region]-[project]/updateVariablesModal.svelte
@@ -76,6 +76,7 @@
         <Selector.Checkbox
             id="secret"
             label="Secret"
+            disabled={selectedVar.secret}
             bind:checked={pair.secret}
             description="If selected, you and your team won't be able to read the values after creation." />
     </Layout.Stack>