From 07ee4f41834a20864625a773bc5bf8cb634f4cf8 Mon Sep 17 00:00:00 2001
From: "courtmanr@gmail.com" <courtmanr@gmail.com>
Date: Fri, 30 May 2025 11:01:16 +0100
Subject: [PATCH] fix: correct test connection success message
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Make success message text dynamic instead of hardcoded
- Test connection now shows "Connection test successful\!"
- Save configuration still shows "Configuration saved. Redirecting..."
- Prevents confusion when testing connection

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

Co-Authored-By: Claude <noreply@anthropic.com>
---
 src/public/setup.html | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/public/setup.html b/src/public/setup.html
index 35772ae88..796e8a446 100644
--- a/src/public/setup.html
+++ b/src/public/setup.html
@@ -276,7 +276,7 @@
             </div>
             <div class="ml-3">
               <h3 class="text-sm font-medium text-green-800 dark:text-green-200">Success!</h3>
-              <div class="mt-2 text-sm text-green-700 dark:text-green-300">Configuration saved. Redirecting to dashboard...</div>
+              <div class="mt-2 text-sm text-green-700 dark:text-green-300" id="success-text">Configuration saved. Redirecting to dashboard...</div>
             </div>
           </div>
         </div>
@@ -325,10 +325,12 @@
       successDiv.classList.add('hidden');
     }
 
-    function showSuccess() {
+    function showSuccess(message = 'Configuration saved. Redirecting to dashboard...') {
       const errorDiv = document.getElementById('error-message');
       const successDiv = document.getElementById('success-message');
+      const successText = document.getElementById('success-text');
       
+      successText.textContent = message;
       errorDiv.classList.add('hidden');
       successDiv.classList.remove('hidden');
     }
@@ -369,7 +371,7 @@
         const result = await response.json();
         
         if (response.ok && result.success) {
-          showSuccess();
+          showSuccess('Connection test successful!');
           setTimeout(() => hideMessages(), 3000);
         } else {
           showError(result.error || 'Connection test failed');