Merge pull request #261 from carverauto/updates/fixing_refresh_rate

cleanup and fixing refresh rate
carverauto · Feb 27, 2025 · b2e68c6 · b2e68c6
2 parents 0fea6af + e333e86
commit b2e68c6
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 8 deletions.
diff --git a/serviceradar-next/src/app/nodes/page.js b/serviceradar-next/src/app/nodes/page.js
@@ -35,8 +35,6 @@ async function fetchNodes() {
 export default async function NodesPage() {
     const initialNodes = await fetchNodes();
 
-    console.log("NodesPage rendered with initialNodes length:", initialNodes.length);
-
     return (
         <div>
             <Suspense fallback={<div className="flex justify-center items-center h-64">

diff --git a/serviceradar-next/src/components/Dashboard.jsx b/serviceradar-next/src/components/Dashboard.jsx
@@ -6,7 +6,7 @@ import { useAPIData } from '@/lib/api';
 
 function Dashboard({ initialData = null }) {
     // Use improved API client with caching - refresh every 30 seconds instead of 10
-    const { data: systemStatus, error, isLoading } = useAPIData('/api/status', initialData, 30000);
+    const { data: systemStatus, error, isLoading } = useAPIData('/api/status', initialData, 10000);
 
     if (isLoading && !systemStatus) {
         return (

diff --git a/serviceradar-next/src/components/NodeList.jsx b/serviceradar-next/src/components/NodeList.jsx
@@ -14,8 +14,7 @@ function NodeList({ initialNodes = [] }) {
   const [sortBy, setSortBy] = useState('name');
   const [sortOrder, setSortOrder] = useState('asc');
 
-  // Use the improved API client with 30 second polling instead of 10
-  const { data: nodes, error, isLoading } = useAPIData('/api/nodes', initialNodes, 30000);
+  const { data: nodes, error, isLoading } = useAPIData('/api/nodes', initialNodes, 10000);
 
   const sortNodesByName = useCallback((a, b) => {
     const aMatch = a.node_id.match(/(\d+)$/);

diff --git a/serviceradar-next/src/components/ServiceSparkline.jsx b/serviceradar-next/src/components/ServiceSparkline.jsx
@@ -7,7 +7,7 @@ import _ from 'lodash';
 import { useAPIData } from '../lib/api';
 
 const MAX_POINTS = 100;
-const POLLING_INTERVAL = 30000; // 30 seconds
+const POLLING_INTERVAL = 10000; // 10 seconds
 
 const ServiceSparkline = React.memo(({ nodeId, serviceName, initialMetrics = [] }) => {
     const { data: metrics, error, isLoading } = useAPIData(
@@ -59,8 +59,6 @@ const ServiceSparkline = React.memo(({ nodeId, serviceName, initialMetrics = []
         setError(error); // Sync error state with useAPIData
     }, [error]);
 
-    console.log("ServiceSparkline rendered", { nodeId, serviceName, metricsLength: metrics?.length, isLoading, error: errorState });
-
     if (isLoading && !processedMetrics.length) {
         return <div className="flex flex-col items-center transition-colors h-8 w-24">
             <div className="h-8 w-24 bg-gray-100 dark:bg-gray-700 animate-pulse rounded"></div>