Skip to content

Commit 07ee4f4

Browse files
rcourtmanclaude
andcommitted
fix: correct test connection success message
- 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>
1 parent bd94648 commit 07ee4f4

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

src/public/setup.html

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@ <h3 class="text-sm font-medium text-red-800 dark:text-red-200">Configuration Err
276276
</div>
277277
<div class="ml-3">
278278
<h3 class="text-sm font-medium text-green-800 dark:text-green-200">Success!</h3>
279-
<div class="mt-2 text-sm text-green-700 dark:text-green-300">Configuration saved. Redirecting to dashboard...</div>
279+
<div class="mt-2 text-sm text-green-700 dark:text-green-300" id="success-text">Configuration saved. Redirecting to dashboard...</div>
280280
</div>
281281
</div>
282282
</div>
@@ -325,10 +325,12 @@ <h3 class="text-sm font-medium text-green-800 dark:text-green-200">Success!</h3>
325325
successDiv.classList.add('hidden');
326326
}
327327

328-
function showSuccess() {
328+
function showSuccess(message = 'Configuration saved. Redirecting to dashboard...') {
329329
const errorDiv = document.getElementById('error-message');
330330
const successDiv = document.getElementById('success-message');
331+
const successText = document.getElementById('success-text');
331332

333+
successText.textContent = message;
332334
errorDiv.classList.add('hidden');
333335
successDiv.classList.remove('hidden');
334336
}
@@ -369,7 +371,7 @@ <h3 class="text-sm font-medium text-green-800 dark:text-green-200">Success!</h3>
369371
const result = await response.json();
370372

371373
if (response.ok && result.success) {
372-
showSuccess();
374+
showSuccess('Connection test successful!');
373375
setTimeout(() => hideMessages(), 3000);
374376
} else {
375377
showError(result.error || 'Connection test failed');

0 commit comments

Comments
 (0)