From adfaeb487d3a480e12b79bcb89ac17f572234b9b Mon Sep 17 00:00:00 2001
From: Giorgio Ravera <giorgio.ravera@gmail.com>
Date: Sun, 31 May 2026 14:03:56 +0200
Subject: [PATCH] improved diagnostic for getting backup list

---
 frontend/js/index.js | 41 +++++++++++++++++++++++++----------------
 1 file changed, 25 insertions(+), 16 deletions(-)

diff --git a/frontend/js/index.js b/frontend/js/index.js
index 72c024a..40cdd88 100644
--- a/frontend/js/index.js
+++ b/frontend/js/index.js
@@ -24,12 +24,12 @@ async function openBackupModal() {
         `;
     }
 
+    // Refresh backup list
     try {
-        const data = await serviceBackupList();
-        renderBackupList(data);
+        const result = await serviceBackupList();
+        renderBackupList(result);
     } catch (err) {
-        console.error(err);
-        showToast("Error loading backups", false);
+        showToast(err?.message || "Error refreshing backup list", false);
     }
 }
 
@@ -281,15 +281,19 @@ const actionHandlers = {
                         ? result.message
                         : 'Backup completed successfully';
             showToast(msg, !result?.partial);
-            // reload list
-            const data = await serviceBackupList();
-            renderBackupList(data);
         } catch (err) {
             showToast(err?.message || "Error performing backup", false);
         } finally {
             label.textContent = originalLabel;
             btn.disabled = false;
         }
+        // Refresh backup list
+        try {
+            const result = await serviceBackupList();
+            renderBackupList(result);
+        } catch (err) {
+            showToast(err?.message || "Error refreshing backup list", false);
+        }
     },
     // Restore Backup
     startRestore: async (e, el) => {
@@ -342,15 +346,17 @@ const actionHandlers = {
                 : 'Backup deleted successfully';
 
             showToast(msg, true);
-
-            // reload list
-            const data = await serviceBackupList();
-            renderBackupList(data);
-
         } catch (err) {
             console.error(err);
             showToast(err?.message || "Error deleting backup", false);
         }
+        // Refresh backup list
+        try {
+            const result = await serviceBackupList();
+            renderBackupList(result);
+        } catch (err) {
+            showToast(err?.message || "Error refreshing backup list", false);
+        }
     },
     refreshBackupList: async () => {
         try {
@@ -415,10 +421,6 @@ const actionHandlers = {
             console.log("Uploaded backup ID:", result?.backup_id);
 
             input.value = '';
-
-            const data = await serviceBackupList();
-            renderBackupList(data);
-
         } catch (err) {
             showToast(err?.message || "Error uploading backup", false);
         } finally {
@@ -427,6 +429,13 @@ const actionHandlers = {
             }
             el.disabled = false;
         }
+        // refresh backup list
+        try {
+            const result = await serviceBackupList();
+            renderBackupList(result);
+        } catch (err) {
+            showToast(err?.message || "Error refreshing backup list", false);
+        }
     },
     openBackupModal,       // managed by boostrap
     closeBackupModal,      // managed by boostrap
-- 
2.47.3