From 7ba40483e057457627533986b5402c983e6920ab Mon Sep 17 00:00:00 2001
From: StarFleetCPTN <starfleetcptn@protonmail.com>
Date: Sat, 19 Apr 2025 17:03:33 -0700
Subject: [PATCH] Add toast deduplication and enhance provider test status
 notifications

---
 components/shared/toast/toast_js.templ | 21 +++++++++++++++++
 components/storage_providers.templ     | 32 ++++++++++++++++++++------
 static/js/storage-providers.js         |  1 +
 3 files changed, 47 insertions(+), 7 deletions(-)

diff --git a/components/shared/toast/toast_js.templ b/components/shared/toast/toast_js.templ
index 599ee2b..62e4c5e 100644
--- a/components/shared/toast/toast_js.templ
+++ b/components/shared/toast/toast_js.templ
@@ -3,7 +3,28 @@ package toast
 templ ShowToastJS() {
 	<script>
 		// Notification system
+		// Global tracking of shown messages to prevent duplicates
+		window.shownToastMessages = window.shownToastMessages || [];
+
 		function showToast(message, type) {
+			// Check if this exact message has been shown in the last 500ms
+			const messageKey = `${message}-${type}`;
+			if (window.shownToastMessages.includes(messageKey)) {
+				console.log(`Preventing duplicate toast: ${messageKey}`);
+				return;
+			}
+
+			// Add to shown messages
+			window.shownToastMessages.push(messageKey);
+			
+			// Remove from tracking after 500ms to allow the same message later if needed
+			setTimeout(() => {
+				const index = window.shownToastMessages.indexOf(messageKey);
+				if (index > -1) {
+					window.shownToastMessages.splice(index, 1);
+				}
+			}, 500);
+
 			const toastContainer = document.getElementById('toast-container');
 			if (!toastContainer) {
 				console.error("Toast container not found!");
diff --git a/components/storage_providers.templ b/components/storage_providers.templ
index da9af00..2928cb3 100644
--- a/components/storage_providers.templ
+++ b/components/storage_providers.templ
@@ -212,15 +212,33 @@ templ StorageProviders(ctx context.Context, data StorageProvidersData) {
 					localStorage.removeItem('showProviderDuplicateToast');
 				}
 				
-				// Keep other status-based toasts if needed
+				// Process URL parameters for toast notifications
 				const urlParams = new URLSearchParams(window.location.search);
+				
+				// Handle combined status and test_status parameters
 				if (urlParams.get('status') === 'created') {
-					showToast('Provider created successfully', 'success');
-				}
-				if (urlParams.get('status') === 'updated') {
-					showToast('Provider updated successfully', 'success');
-				}
-				if (urlParams.get('error')) {
+					const testStatus = urlParams.get('test_status');
+					const errorMsg = urlParams.get('error');
+					
+					if (testStatus === 'success') {
+						showToast('Provider created and tested successfully', 'success');
+					} else if (testStatus === 'failed' && errorMsg) {
+						showToast(`Provider created but test failed: ${errorMsg}`, 'error');
+					} else {
+						showToast('Provider created successfully', 'success');
+					}
+				} else if (urlParams.get('status') === 'updated') {
+					const testStatus = urlParams.get('test_status');
+					const errorMsg = urlParams.get('error');
+					
+					if (testStatus === 'success') {
+						showToast('Provider updated and tested successfully', 'success');
+					} else if (testStatus === 'failed' && errorMsg) {
+						showToast(`Provider updated but test failed: ${errorMsg}`, 'error');
+					} else {
+						showToast('Provider updated successfully', 'success');
+					}
+				} else if (urlParams.get('error')) {
 					showToast(urlParams.get('error'), 'error');
 				}
 			});
diff --git a/static/js/storage-providers.js b/static/js/storage-providers.js
index 1b1f68b..d22cb37 100644
--- a/static/js/storage-providers.js
+++ b/static/js/storage-providers.js
@@ -1,4 +1,5 @@
 document.addEventListener('DOMContentLoaded', () => {
+
   // Show test modal when the test button is clicked
   document.querySelectorAll('.test-provider-btn').forEach(button => {
     button.addEventListener('click', async (e) => {