gsadmin/pulse
1
0
Fork 0
mirror of https://github.com/rcourtman/Pulse.git synced 2026-09-23 03:33:53 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: merge temperature data from WebSocket state into Settings node list

Browse Source 
- Settings page now merges live temperature data from WebSocket state
- Temperature badge will now appear when SSH monitoring is active
- Fix missing onMount import in GuestRow.tsx

The Settings page loads node config from /api/config/nodes which doesn't
include runtime temperature data. This change merges temperature from the
WebSocket state (which has live monitoring data) so the temperature badge
displays correctly.
This commit is contained in:
rcourtman
2025-10-01 08:21:50 +00:00
parent 2f8cf462bb
commit 1ef938071a
2 changed files with 19 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
frontend-modern/src/components/Dashboard/GuestRow.tsx
+1 -1
View File
@@ -1,4 +1,4 @@
import { Show, createMemo, createSignal, createEffect } from 'solid-js';
import { Show, createMemo, createSignal, createEffect, onMount } from 'solid-js';
import type { VM, Container } from '@/types/api';
import { formatBytes, formatUptime } from '@/utils/format';
import { MetricBar } from './MetricBar';
frontend-modern/src/components/Settings/Settings.tsx
+18 -8
View File
@@ -258,14 +258,24 @@ const Settings: Component = () => {
const loadNodes = async () => {
try {
const nodesList = await NodesAPI.getNodes();
// Add status and other UI fields
const nodesWithStatus = nodesList.map((node) => ({
...node,
// Use the hasPassword/hasToken from the API if available, otherwise check local fields
hasPassword: node.hasPassword ?? !!node.password,
hasToken: node.hasToken ?? !!node.tokenValue,
status: node.status || ('pending' as const),
}));
// Merge temperature data from WebSocket state
const currentState = state();
const nodesWithStatus = nodesList.map((node) => {
// Find matching node in state to get temperature data
const stateNode =
currentState?.pveNodes?.find((n) => n.id === node.id || n.name === node.name) ||
currentState?.pbsNodes?.find((n) => n.id === node.id || n.name === node.name);
return {
...node,
// Use the hasPassword/hasToken from the API if available, otherwise check local fields
hasPassword: node.hasPassword ?? !!node.password,
hasToken: node.hasToken ?? !!node.tokenValue,
status: node.status || ('pending' as const),
// Merge temperature data from state
temperature: stateNode?.temperature || node.temperature,
};
});
setNodes(nodesWithStatus);
} catch (error) {
console.error('Failed to load nodes:', error);