From 940a253dc8a6f4e6598a641dad7f37cf64e0503b Mon Sep 17 00:00:00 2001 From: Pulse Monitor Date: Sat, 30 Aug 2025 17:12:33 +0000 Subject: [PATCH] fix: acknowledge alert button not updating count (addresses #380) - Fixed reactivity issue with alertStats computation in SolidJS - Properly access store values to ensure count updates when alerts are acknowledged - Changed from Object.values() to mapping over keys for proper store reactivity --- frontend-modern/src/pages/Alerts.tsx | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/frontend-modern/src/pages/Alerts.tsx b/frontend-modern/src/pages/Alerts.tsx index 0dc53d839..826ac81f3 100644 --- a/frontend-modern/src/pages/Alerts.tsx +++ b/frontend-modern/src/pages/Alerts.tsx @@ -732,7 +732,9 @@ function OverviewTab(props: { // Get alert stats from actual active alerts const alertStats = createMemo(() => { - const alerts = Object.values(props.activeAlerts); + // Access the store properly for reactivity + const alertIds = Object.keys(props.activeAlerts); + const alerts = alertIds.map(id => props.activeAlerts[id]); return { active: alerts.filter(a => !a.acknowledged).length, acknowledged: alerts.filter(a => a.acknowledged).length,