From 599b943bdaef27522eb9a06c745f106142efdbcb Mon Sep 17 00:00:00 2001
From: "courtmanr@gmail.com" <courtmanr@gmail.com>
Date: Tue, 20 May 2025 15:14:34 +0100
Subject: [PATCH] refactor(backups): Remove redundant health status indicator
 from table

---
 src/public/index.html       |  1 -
 src/public/js/ui/backups.js | 10 ----------
 2 files changed, 11 deletions(-)

diff --git a/src/public/index.html b/src/public/index.html
index 3acd6de0a..314401b5d 100644
--- a/src/public/index.html
+++ b/src/public/index.html
@@ -428,7 +428,6 @@
           <table id="backups-overview-table" class="min-w-full divide-y divide-gray-200 dark:divide-gray-700 text-sm">
             <thead class="sticky top-0 z-10 bg-gray-100 dark:bg-gray-800">
               <tr class="text-xs font-medium tracking-wider text-left text-gray-600 uppercase bg-gray-100 dark:bg-gray-700 dark:text-gray-300 border-b border-gray-300 dark:border-gray-600">
-                <th scope="col" class="sortable p-1 px-2 text-center cursor-pointer select-none whitespace-nowrap w-[30px]" data-sort="backupHealthStatus"></th>
                 <th scope="col" class="sortable p-1 px-2 cursor-pointer select-none whitespace-nowrap" data-sort="guestName">Guest Name</th>
                 <th scope="col" class="sortable p-1 px-2 cursor-pointer select-none whitespace-nowrap" data-sort="guestId">ID</th>
                 <th scope="col" class="sortable p-1 px-2 cursor-pointer select-none whitespace-nowrap" data-sort="guestType">Type</th>
diff --git a/src/public/js/ui/backups.js b/src/public/js/ui/backups.js
index 8e6e55fc1..53473d4bc 100644
--- a/src/public/js/ui/backups.js
+++ b/src/public/js/ui/backups.js
@@ -196,15 +196,6 @@ PulseApp.ui.backups = (() => {
             ? PulseApp.utils.formatPbsTimestamp(guestStatus.latestBackupTime)
             : '<span class="text-gray-400">No backups found</span>';
 
-        let healthIndicator = '';
-        switch (guestStatus.backupHealthStatus) {
-            case 'ok': healthIndicator = '<span class="text-green-600 dark:text-green-400" title="OK">●</span>'; break;
-            case 'stale': healthIndicator = '<span class="text-yellow-600 dark:text-yellow-400" title="Stale">●</span>'; break;
-            case 'failed': healthIndicator = '<span class="text-red-600 dark:text-red-400 font-bold" title="Failed">✖</span>'; break;
-            case 'old': healthIndicator = '<span class="text-orange-600 dark:text-orange-400" title="Old">●</span>'; break;
-            case 'none': healthIndicator = '<span class="text-gray-400 dark:text-gray-500" title="None">-</span>'; break;
-        }
-
         const typeIconClass = guestStatus.guestType === 'VM'
             ? 'vm-icon bg-blue-100 dark:bg-blue-900/50 text-blue-700 dark:text-blue-300 px-1.5 py-0.5 font-medium'
             : 'ct-icon bg-green-100 dark:bg-green-900/50 text-green-700 dark:text-green-300 px-1.5 py-0.5 font-medium';
@@ -233,7 +224,6 @@ PulseApp.ui.backups = (() => {
         sevenDayDots += '</div>';
 
         row.innerHTML = `
-            <td class="p-1 px-2 whitespace-nowrap text-center">${healthIndicator}</td>
             <td class="p-1 px-2 whitespace-nowrap font-medium text-gray-900 dark:text-gray-100" title="${guestStatus.guestName}">${guestStatus.guestName}</td>
             <td class="p-1 px-2 text-gray-500 dark:text-gray-400">${guestStatus.guestId}</td>
             <td class="p-1 px-2">${typeIcon}</td>