Removed development logging.

Fixed all warnings.
Fixed base.
2025-12-27 15:19:09 +00:00 · 2025-12-27 15:06:02 +00:00 · 2025-12-27 14:32:00 +00:00
59 changed files with 250 additions and 318 deletions
--- a/public/electron.js
+++ b/public/electron.js
@ -87,6 +87,5 @@ ipcMain.handle('auth-session-clear', async () => {

 // IPC handler for opening external URLs
 ipcMain.handle('open-external-url', (event, url) => {
-  console.log('Opening external url...')
  shell.openExternal(url)
 })
--- a/public/mainWindow.js
+++ b/public/mainWindow.js
@ -155,7 +155,6 @@ export function setupMainWindowAppEvents(app) {

  app.on('open-url', (event, url) => {
    event.preventDefault()
-    console.log('App opened with URL:', url)
    if (url.startsWith('farmcontrol://app')) {
      // Extract the path/query after 'farmcontrol://app'
      const redirectPath = url.replace('farmcontrol://app', '') || '/'
--- a/public/manifest.json
+++ b/public/manifest.json
@ -1,6 +1,6 @@
 {
-  "short_name": "React App",
-  "name": "Create React App Sample",
+  "short_name": "Farm Control",
+  "name": "Farm Control",
  "icons": [
    {
      "src": "favicon.ico",
--- a/src/components/Dashboard/Developer/ApiContextDebug.jsx
+++ b/src/components/Dashboard/Developer/ApiContextDebug.jsx
@ -17,6 +17,10 @@ import CloudIcon from '../../Icons/CloudIcon.jsx'
 import { ApiServerContext } from '../context/ApiServerContext.jsx'
 import BoolDisplay from '../common/BoolDisplay.jsx'
 import InfoCollapse from '../common/InfoCollapse.jsx'
+import loglevel from 'loglevel'
+import config from '../../../config'
+const logger = loglevel.getLogger('ApiContextDebug')
+logger.setLevel(config.logLevel)

 const { Text, Paragraph } = Typography

@ -140,11 +144,10 @@ const ApiContextDebug = () => {
      )
      msgApi.destroy()
      msgApi.success('fetchObject test completed')
-      console.log('fetchObject result:', result)
    } catch (err) {
      msgApi.destroy()
      msgApi.error('fetchObject test failed')
-      console.error('fetchObject error:', err)
+      logger.error('fetchObject error:', err)
    }
  }

@ -157,11 +160,10 @@ const ApiContextDebug = () => {
      })
      msgApi.destroy()
      msgApi.success('fetchObjects test completed')
-      console.log('fetchObjects result:', result)
    } catch (err) {
      msgApi.destroy()
      msgApi.error('fetchObjects test failed')
-      console.error('fetchObjects error:', err)
+      logger.error('fetchObjects error:', err)
    }
  }

@ -171,7 +173,7 @@ const ApiContextDebug = () => {
      msgApi.success('Lock command sent')
    } catch (err) {
      msgApi.error('Lock command failed')
-      console.error('Lock error:', err)
+      logger.error('Lock error:', err)
    }
  }

@ -181,7 +183,7 @@ const ApiContextDebug = () => {
      msgApi.success('Unlock command sent')
    } catch (err) {
      msgApi.error('Unlock command failed')
-      console.error('Unlock error:', err)
+      logger.error('Unlock error:', err)
    }
  }

@ -194,11 +196,10 @@ const ApiContextDebug = () => {
      )
      msgApi.destroy()
      msgApi.success('fetchObjectLock test completed')
-      console.log('fetchObjectLock result:', result)
    } catch (err) {
      msgApi.destroy()
      msgApi.error('fetchObjectLock test failed')
-      console.error('fetchObjectLock error:', err)
+      logger.error('fetchObjectLock error:', err)
    }
  }

@ -216,11 +217,10 @@ const ApiContextDebug = () => {
      )
      msgApi.destroy()
      msgApi.success('updateObject test completed')
-      console.log('updateObject result:', result)
    } catch (err) {
      msgApi.destroy()
      msgApi.error('updateObject test failed')
-      console.error('updateObject error:', err)
+      logger.error('updateObject error:', err)
    }
  }

@ -234,11 +234,10 @@ const ApiContextDebug = () => {
      const result = await createObject(testInputs.objectType, testData)
      msgApi.destroy()
      msgApi.success('createObject test completed')
-      console.log('createObject result:', result)
    } catch (err) {
      msgApi.destroy()
      msgApi.error('createObject test failed')
-      console.error('createObject error:', err)
+      logger.error('createObject error:', err)
    }
  }

@ -251,11 +250,10 @@ const ApiContextDebug = () => {
      )
      msgApi.destroy()
      msgApi.success('deleteObject test completed')
-      console.log('deleteObject result:', result)
    } catch (err) {
      msgApi.destroy()
      msgApi.error('deleteObject test failed')
-      console.error('deleteObject error:', err)
+      logger.error('deleteObject error:', err)
    }
  }

@ -270,11 +268,10 @@ const ApiContextDebug = () => {
      const result = await fetchObjectsByProperty(testInputs.objectType, params)
      msgApi.destroy()
      msgApi.success('fetchObjectsByProperty test completed')
-      console.log('fetchObjectsByProperty result:', result)
    } catch (err) {
      msgApi.destroy()
      msgApi.error('fetchObjectsByProperty test failed')
-      console.error('fetchObjectsByProperty error:', err)
+      logger.error('fetchObjectsByProperty error:', err)
    }
  }

@ -284,11 +281,10 @@ const ApiContextDebug = () => {
      const result = await fetchSpotlightData(testInputs.query)
      msgApi.destroy()
      msgApi.success('fetchSpotlightData test completed')
-      console.log('fetchSpotlightData result:', result)
    } catch (err) {
      msgApi.destroy()
      msgApi.error('fetchSpotlightData test failed')
-      console.error('fetchSpotlightData error:', err)
+      logger.error('fetchSpotlightData error:', err)
    }
  }

@ -305,7 +301,7 @@ const ApiContextDebug = () => {
    } catch (err) {
      msgApi.destroy()
      msgApi.error('fetchFileContent test failed')
-      console.error('fetchFileContent error:', err)
+      logger.error('fetchFileContent error:', err)
    }
  }

@ -325,13 +321,12 @@ const ApiContextDebug = () => {
          } else {
            msgApi.error('fetchTemplatePreview test failed')
          }
-          console.log('fetchTemplatePreview result:', result)
        }
      )
    } catch (err) {
      msgApi.destroy()
      msgApi.error('fetchTemplatePreview test failed')
-      console.error('fetchTemplatePreview error:', err)
+      logger.error('fetchTemplatePreview error:', err)
    }
  }

@ -341,11 +336,10 @@ const ApiContextDebug = () => {
      const result = await fetchNotes(testInputs.parentId)
      msgApi.destroy()
      msgApi.success('fetchNotes test completed')
-      console.log('fetchNotes result:', result)
    } catch (err) {
      msgApi.destroy()
      msgApi.error('fetchNotes test failed')
-      console.error('fetchNotes error:', err)
+      logger.error('fetchNotes error:', err)
    }
  }

@ -360,12 +354,11 @@ const ApiContextDebug = () => {
        } else {
          msgApi.error('fetchHostOTP test failed')
        }
-        console.log('fetchHostOTP result:', result)
      })
    } catch (err) {
      msgApi.destroy()
      msgApi.error('fetchHostOTP test failed')
-      console.error('fetchHostOTP error:', err)
+      logger.error('fetchHostOTP error:', err)
    }
  }

@ -384,20 +377,19 @@ const ApiContextDebug = () => {
          } else {
            msgApi.error('sendObjectAction test failed')
          }
-          console.log('sendObjectAction result:', result)
        }
      )
    } catch (err) {
      msgApi.destroy()
      msgApi.error('sendObjectAction test failed')
-      console.error('sendObjectAction error:', err)
+      logger.error('sendObjectAction error:', err)
    }
  }

  const testSubscribeToObjectUpdates = () => {
    try {
      const callback = (data) => {
-        console.log('Object update received:', data)
+        logger.debug('Object update received:', data)
      }
      const unsubscribe = subscribeToObjectUpdates(
        testInputs.objectId,
@ -405,25 +397,25 @@ const ApiContextDebug = () => {
        callback
      )
      msgApi.success('Subscribed to object updates')
-      console.log('Subscribed to object updates for test-id')
+      logger.debug('Subscribed to object updates for test-id')

      // Store unsubscribe function for cleanup
      setTimeout(() => {
        if (unsubscribe) {
          unsubscribe()
-          console.log('Unsubscribed from object updates')
+          logger.debug('Unsubscribed from object updates')
        }
      }, 10000) // Auto-unsubscribe after 10 seconds
    } catch (err) {
      msgApi.error('Subscribe to object updates failed')
-      console.error('Subscribe error:', err)
+      logger.error('Subscribe error:', err)
    }
  }

  const testSubscribeToObjectEvent = () => {
    try {
      const callback = (event) => {
-        console.log('Object event received:', event)
+        logger.debug('Object event received:', event)
      }
      const unsubscribe = subscribeToObjectEvent(
        testInputs.objectId,
@ -432,50 +424,50 @@ const ApiContextDebug = () => {
        callback
      )
      msgApi.success('Subscribed to object events')
-      console.log('Subscribed to object events for test-id')
+      logger.debug('Subscribed to object events for test-id')

      // Store unsubscribe function for cleanup
      setTimeout(() => {
        if (unsubscribe) {
          unsubscribe()
-          console.log('Unsubscribed from object events')
+          logger.debug('Unsubscribed from object events')
        }
      }, 10000) // Auto-unsubscribe after 10 seconds
    } catch (err) {
      msgApi.error('Subscribe to object events failed')
-      console.error('Subscribe error:', err)
+      logger.error('Subscribe error:', err)
    }
  }

  const testSubscribeToObjectTypeUpdates = () => {
    try {
      const callback = (data) => {
-        console.log('Object type update received:', data)
+        logger.debug('Object type update received:', data)
      }
      const unsubscribe = subscribeToObjectTypeUpdates(
        testInputs.objectType,
        callback
      )
      msgApi.success('Subscribed to object type updates')
-      console.log('Subscribed to object type updates for user')
+      logger.debug('Subscribed to object type updates for user')

      // Store unsubscribe function for cleanup
      setTimeout(() => {
        if (unsubscribe) {
          unsubscribe()
-          console.log('Unsubscribed from object type updates')
+          logger.debug('Unsubscribed from object type updates')
        }
      }, 10000) // Auto-unsubscribe after 10 seconds
    } catch (err) {
      msgApi.error('Subscribe to object type updates failed')
-      console.error('Subscribe error:', err)
+      logger.error('Subscribe error:', err)
    }
  }

  const testSubscribeToObjectLock = () => {
    try {
      const callback = (lockData) => {
-        console.log('Object lock update received:', lockData)
+        logger.debug('Object lock update received:', lockData)
      }
      const unsubscribe = subscribeToObjectLock(
        testInputs.objectId,
@ -483,18 +475,18 @@ const ApiContextDebug = () => {
        callback
      )
      msgApi.success('Subscribed to object lock updates')
-      console.log('Subscribed to object lock updates for test-id')
+      logger.debug('Subscribed to object lock updates for test-id')

      // Store unsubscribe function for cleanup
      setTimeout(() => {
        if (unsubscribe) {
          unsubscribe()
-          console.log('Unsubscribed from object lock updates')
+          logger.debug('Unsubscribed from object lock updates')
        }
      }, 10000) // Auto-unsubscribe after 10 seconds
    } catch (err) {
      msgApi.error('Subscribe to object lock updates failed')
-      console.error('Subscribe error:', err)
+      logger.error('Subscribe error:', err)
    }
  }

--- a/src/components/Dashboard/Inventory/FilamentStocks/LoadFilamentStock.jsx
+++ b/src/components/Dashboard/Inventory/FilamentStocks/LoadFilamentStock.jsx
@ -67,7 +67,6 @@ const LoadFilamentStock = ({
        'printer',
        'filamentSensor',
        (event) => {
-          console.log('filamentSensor', event.data)
          setFilamentSensorDetected(event.data.detected)
        }
      )
@ -76,7 +75,7 @@ const LoadFilamentStock = ({
        if (filamentStockEventUnsubscribe) filamentStockEventUnsubscribe()
      }
    }
-  }, [printer?._id, connected])
+  }, [printer?._id, connected, subscribeToObjectEvent])

  useEffect(() => {
    // Validate form fields
--- a/src/components/Dashboard/Inventory/FilamentStocks/UnloadFilamentStock.jsx
+++ b/src/components/Dashboard/Inventory/FilamentStocks/UnloadFilamentStock.jsx
@ -60,7 +60,7 @@ const UnloadFilamentStock = ({ onOk, reset, printer = null }) => {
        if (filamentStockEventUnsubscribe) filamentStockEventUnsubscribe()
      }
    }
-  }, [printer?._id, connected])
+  }, [printer?._id, connected, subscribeToObjectEvent])

  useEffect(() => {
    if (reset) {
--- a/src/components/Dashboard/Inventory/PartStocks/PartStockInfo.jsx
+++ b/src/components/Dashboard/Inventory/PartStocks/PartStockInfo.jsx
@ -145,7 +145,6 @@ const PartStockInfo = () => {
                  style={{ height: '100%' }}
                  ref={objectFormRef}
                  onStateChange={(state) => {
-                    console.log('Got edit form state change', state)
                    setEditFormState((prev) => ({ ...prev, ...state }))
                  }}
                >
--- a/src/components/Dashboard/Inventory/StockAudits/StockAuditInfo.jsx
+++ b/src/components/Dashboard/Inventory/StockAudits/StockAuditInfo.jsx
@ -145,7 +145,6 @@ const StockAuditInfo = () => {
                  style={{ height: '100%' }}
                  ref={objectFormRef}
                  onStateChange={(state) => {
-                    console.log('Got edit form state change', state)
                    setEditFormState((prev) => ({ ...prev, ...state }))
                  }}
                >
--- a/src/components/Dashboard/Management/DocumentJobs/NewDocumentJob.jsx
+++ b/src/components/Dashboard/Management/DocumentJobs/NewDocumentJob.jsx
@ -60,9 +60,6 @@ const NewDocumentJob = ({ onOk, defaultValues = {} }) => {
                <TemplatePreview
                  objectData={objectData?.object}
                  documentTemplate={objectData?.documentTemplate}
-                  onPreviewMessage={(message) => {
-                    console.log(message)
-                  }}
                />
              </div>
            }
--- a/src/components/Dashboard/Management/DocumentTemplates/DocumentTemplateDesign.jsx
+++ b/src/components/Dashboard/Management/DocumentTemplates/DocumentTemplateDesign.jsx
@ -136,7 +136,6 @@ const DocumentTemplateDesign = () => {
              style={{ height: '100%' }}
              ref={objectFormRef}
              onStateChange={(state) => {
-                console.log('Got edit form state change', state)
                setEditFormState((prev) => ({ ...prev, ...state }))
              }}
            >
--- a/src/components/Dashboard/Management/DocumentTemplates/DocumentTemplateInfo.jsx
+++ b/src/components/Dashboard/Management/DocumentTemplates/DocumentTemplateInfo.jsx
@ -147,7 +147,6 @@ const DocumentTemplateInfo = () => {
                  style={{ height: '100%' }}
                  ref={objectFormRef}
                  onStateChange={(state) => {
-                    console.log('Got edit form state change', state)
                    setEditFormState((prev) => ({ ...prev, ...state }))
                  }}
                >
--- a/src/components/Dashboard/Management/Files/FileInfo.jsx
+++ b/src/components/Dashboard/Management/Files/FileInfo.jsx
@ -151,7 +151,6 @@ const FileInfo = () => {
                  style={{ height: '100%' }}
                  ref={objectFormRef}
                  onStateChange={(state) => {
-                    console.log(state)
                    setEditFormState((prev) => ({ ...prev, ...state }))
                  }}
                >
--- a/src/components/Dashboard/Management/Hosts/HostOtp.jsx
+++ b/src/components/Dashboard/Management/Hosts/HostOtp.jsx
@ -1,5 +1,5 @@
 import PropTypes from 'prop-types'
-import { useContext, useEffect, useState, useRef } from 'react'
+import { useContext, useEffect, useState, useRef, useCallback } from 'react'
 import { Input, Result, Typography, Flex, Progress } from 'antd'
 import { LoadingOutlined } from '@ant-design/icons'
 import { ApiServerContext } from '../../context/ApiServerContext'
@ -17,7 +17,7 @@ const HostOTP = ({ id }) => {
  const [initialized, setInitialized] = useState(false)
  const intervalRef = useRef(null)

-  const fetchNewOTP = () => {
+  const fetchNewOTP = useCallback(() => {
    setLoading(true)
    setHostObject(null) // Reset to show loading
    fetchHostOTP(id, (hostOTPObject) => {
@ -33,14 +33,14 @@ const HostOTP = ({ id }) => {
        setTotalTime(remaining)
      }
    })
-  }
+  }, [id, fetchHostOTP])

  useEffect(() => {
    if (hostObject === null && initialized == false) {
      setInitialized(true)
      fetchNewOTP()
    }
-  }, [id])
+  }, [id, fetchNewOTP, initialized, hostObject])

  useEffect(() => {
    if (hostObject && timeRemaining > 0) {
@ -62,7 +62,7 @@ const HostOTP = ({ id }) => {
        }
      }
    }
-  }, [hostObject, timeRemaining])
+  }, [hostObject, timeRemaining, fetchNewOTP])

  // Clean up interval on unmount
  useEffect(() => {
--- a/src/components/Dashboard/Production/GCodeFiles/GCodeFileInfo.jsx
+++ b/src/components/Dashboard/Production/GCodeFiles/GCodeFileInfo.jsx
@ -147,7 +147,6 @@ const GCodeFileInfo = () => {
                style={{ height: '100%' }}
                ref={objectFormRef}
                onStateChange={(state) => {
-                  console.log('Got edit form state change', state)
                  setEditFormState((prev) => ({ ...prev, ...state }))
                }}
              >
--- a/src/components/Dashboard/Production/Jobs/DeployJob.jsx
+++ b/src/components/Dashboard/Production/Jobs/DeployJob.jsx
@ -1,4 +1,4 @@
-import { useState, useContext, useEffect } from 'react'
+import { useState, useContext, useEffect, useCallback } from 'react'
 import PropTypes from 'prop-types'
 import WizardView from '../../common/WizardView'
 import { ApiServerContext } from '../../context/ApiServerContext'
@ -15,7 +15,7 @@ const DeployJob = ({ onOk, objectData = undefined }) => {
  const [subJobsCount, setSubJobsCount] = useState(999)
  const { sendObjectAction, fetchObjects } = useContext(ApiServerContext)

-  const handleDeploy = async () => {
+  const handleDeploy = useCallback(async () => {
    setDeployLoading(true)
    var hasMore = true
    var currentPage = 1
@ -34,29 +34,22 @@ const DeployJob = ({ onOk, objectData = undefined }) => {
    }
    setSubJobsCount(subJobs.length)
    subJobs.forEach((subJob) => {
-      console.log(
-        'Deploying subjob:',
-        subJob._id,
-        'to printer:',
-        subJob.printer._id
-      )
      sendObjectAction(
        subJob.printer._id,
        'printer',
        { type: 'deploy', data: subJob },
-        (result) => {
-          console.log('result', result)
+        () => {
          setDeployedSubJobsCount((prev) => prev + 1)
        }
      )
    })
-  }
+  }, [job._id, sendObjectAction, fetchObjects])

  useEffect(() => {
    if (deployedSubJobsCount == subJobsCount && deployLoading == true) {
      onOk()
    }
-  }, [deployedSubJobsCount, subJobsCount, deployLoading])
+  }, [deployedSubJobsCount, subJobsCount, deployLoading, onOk])

  const steps = [
    {
--- a/src/components/Dashboard/Production/Printers/ControlPrinter.jsx
+++ b/src/components/Dashboard/Production/Printers/ControlPrinter.jsx
@ -324,7 +324,6 @@ const ControlPrinter = () => {
                          type='printer'
                          ref={objectFormRef}
                          onStateChange={(state) => {
-                            console.log('Got edit form state change', state)
                            setEditFormState((prev) => ({ ...prev, ...state }))
                          }}
                        >
--- a/src/components/Dashboard/Production/Printers/PrinterInfo.jsx
+++ b/src/components/Dashboard/Production/Printers/PrinterInfo.jsx
@ -53,7 +53,6 @@ const PrinterInfo = () => {
    },
    edit: () => {
      objectFormRef?.current.startEditing()
-      console.log('CALLING START EDITING')
      return false
    },
    cancelEdit: () => {
@ -143,7 +142,6 @@ const PrinterInfo = () => {
                  style={{ height: '100%' }}
                  ref={objectFormRef}
                  onStateChange={(state) => {
-                    console.log('Got edit form state change', state)
                    setEditFormState((prev) => ({ ...prev, ...state }))
                  }}
                >
--- a/src/components/Dashboard/common/AlertsDisplay.jsx
+++ b/src/components/Dashboard/common/AlertsDisplay.jsx
@ -162,9 +162,7 @@ const AlertsDisplay = ({
        icon={getAlertIcon(alert.type, alert.priority)}
        showIcon
        closable={showDismiss && alert.canDismiss}
-        onClose={() => {
-          console.log('Closing alert:', alert._id)
-        }}
+        onClose={() => {}}
        action={
          showActions ? (
            <Dropdown menu={menu} on>
--- a/src/components/Dashboard/common/DashboardNavigation.jsx
+++ b/src/components/Dashboard/common/DashboardNavigation.jsx
@ -1,5 +1,5 @@
 // DashboardNavigation.js
-import { useContext, useEffect, useState } from 'react'
+import { useContext, useEffect, useState, useMemo } from 'react'
 import {
  Menu,
  Flex,
@ -59,28 +59,31 @@ const DashboardNavigation = () => {
  const isMobile = useMediaQuery({ maxWidth: 768 })
  const { platform, isElectron, isFullScreen } = useContext(ElectronContext)

-  const mainMenuItems = [
-    {
-      key: 'production',
-      label: 'Production',
-      icon: <ProductionIcon />
-    },
-    {
-      key: 'inventory',
-      label: 'Inventory',
-      icon: <InventoryIcon />
-    },
-    {
-      key: 'finance',
-      label: 'Finance',
-      icon: <FinanceIcon />
-    },
-    {
-      key: 'management',
-      label: 'Management',
-      icon: <SettingsIcon />
-    }
-  ]
+  const mainMenuItems = useMemo(
+    () => [
+      {
+        key: 'production',
+        label: 'Production',
+        icon: <ProductionIcon />
+      },
+      {
+        key: 'inventory',
+        label: 'Inventory',
+        icon: <InventoryIcon />
+      },
+      {
+        key: 'finance',
+        label: 'Finance',
+        icon: <FinanceIcon />
+      },
+      {
+        key: 'management',
+        label: 'Management',
+        icon: <SettingsIcon />
+      }
+    ],
+    []
+  )

  const userMenuItems = {
    items: [
@ -119,7 +122,7 @@ const DashboardNavigation = () => {
      )
      setSelectedKey(pathParts[1]) // Return the section (production/management)
    }
-  }, [location.pathname])
+  }, [location.pathname, mainMenuItems])

  useEffect(() => {
    if (connecting == true) {
--- a/src/components/Dashboard/common/DataTree.jsx
+++ b/src/components/Dashboard/common/DataTree.jsx
@ -1,6 +1,6 @@
 import PropTypes from 'prop-types'
 import { Tree, Typography, Space, Tag } from 'antd'
-import { useState, useMemo } from 'react'
+import { useState, useMemo, useCallback } from 'react'
 import XMarkIcon from '../../Icons/XMarkIcon'
 import QuestionCircleIcon from '../../Icons/QuestionCircleIcon'
 import JsonStringIcon from '../../Icons/JsonStringIcon'
@ -48,7 +48,7 @@ const DataTree = ({
  }

  // Function to format value for display
-  const formatValue = (value) => {
+  const formatValue = useCallback((value) => {
    if (value === null) return 'null'
    if (value === undefined) return 'undefined'
    if (typeof value === 'boolean') return value.toString()
@ -63,10 +63,10 @@ const DataTree = ({
      return `Object (${keys.length} properties)`
    }
    return String(value)
-  }
+  }, [])

  // Function to get raw value for copying
-  const getCopyValue = (value) => {
+  const getCopyValue = useCallback((value) => {
    if (value === null) return 'null'
    if (value === undefined) return 'undefined'
    if (typeof value === 'boolean') return value.toString()
@ -75,64 +75,67 @@ const DataTree = ({
    if (Array.isArray(value)) return JSON.stringify(value, null, 2)
    if (typeof value === 'object') return JSON.stringify(value, null, 2)
    return String(value)
-  }
+  }, [])

  // Recursive function to convert JSON to tree data
-  const convertToTreeData = (obj, key = 'root', path = '') => {
-    const currentPath = path ? `${path}.${key}` : key
-    const dataInfo = getDataTypeInfo(obj)
+  const convertToTreeData = useCallback(
+    (obj, key = 'root', path = '') => {
+      const currentPath = path ? `${path}.${key}` : key
+      const dataInfo = getDataTypeInfo(obj)

-    const node = {
-      title: (
-        <Space size='small'>
-          <Tag color={dataInfo.color} size='small' style={{ margin: 0 }}>
-            {dataInfo.icon}
-          </Tag>
-          <Text strong>{key}</Text>
-          {showKeyCopy && (
-            <CopyButton text={key} tooltip={`Copy key: ${key}`} />
-          )}
-          <Text type='secondary'>({dataInfo.type})</Text>
-          {dataInfo.type !== 'object' && dataInfo.type !== 'array' && (
-            <>
-              <Text code>{formatValue(obj)}</Text>
-              {showValueCopy && (
+      const node = {
+        title: (
+          <Space size='small'>
+            <Tag color={dataInfo.color} size='small' style={{ margin: 0 }}>
+              {dataInfo.icon}
+            </Tag>
+            <Text strong>{key}</Text>
+            {showKeyCopy && (
+              <CopyButton text={key} tooltip={`Copy key: ${key}`} />
+            )}
+            <Text type='secondary'>({dataInfo.type})</Text>
+            {dataInfo.type !== 'object' && dataInfo.type !== 'array' && (
+              <>
+                <Text code>{formatValue(obj)}</Text>
+                {showValueCopy && (
+                  <CopyButton
+                    text={getCopyValue(obj)}
+                    tooltip={`Copy ${dataInfo.type} value`}
+                  />
+                )}
+              </>
+            )}
+            {(dataInfo.type === 'object' || dataInfo.type === 'array') &&
+              showValueCopy && (
                <CopyButton
                  text={getCopyValue(obj)}
-                  tooltip={`Copy ${dataInfo.type} value`}
+                  tooltip={`Copy ${dataInfo.type} as JSON`}
                />
              )}
-            </>
-          )}
-          {(dataInfo.type === 'object' || dataInfo.type === 'array') &&
-            showValueCopy && (
-              <CopyButton
-                text={getCopyValue(obj)}
-                tooltip={`Copy ${dataInfo.type} as JSON`}
-              />
-            )}
-        </Space>
-      ),
-      key: currentPath,
-      value: obj,
-      path: currentPath
-    }
-
-    // Add children for objects and arrays
-    if (typeof obj === 'object' && obj !== null) {
-      if (Array.isArray(obj)) {
-        node.children = obj.map((item, index) =>
-          convertToTreeData(item, `[${index}]`, currentPath)
-        )
-      } else {
-        node.children = Object.entries(obj).map(([childKey, childValue]) =>
-          convertToTreeData(childValue, childKey, currentPath)
-        )
+          </Space>
+        ),
+        key: currentPath,
+        value: obj,
+        path: currentPath
      }
-    }

-    return node
-  }
+      // Add children for objects and arrays
+      if (typeof obj === 'object' && obj !== null) {
+        if (Array.isArray(obj)) {
+          node.children = obj.map((item, index) =>
+            convertToTreeData(item, `[${index}]`, currentPath)
+          )
+        } else {
+          node.children = Object.entries(obj).map(([childKey, childValue]) =>
+            convertToTreeData(childValue, childKey, currentPath)
+          )
+        }
+      }
+
+      return node
+    },
+    [showValueCopy, getCopyValue, formatValue, showKeyCopy]
+  )

  // Convert data to tree structure
  const treeData = useMemo(() => {
@ -171,7 +174,7 @@ const DataTree = ({
        }
      ]
    }
-  }, [data])
+  }, [data, showValueCopy, convertToTreeData, getCopyValue, formatValue])

  // Handle node selection
  const handleSelect = (selectedKeys, { selected, selectedNodes }) => {
--- a/src/components/Dashboard/common/FilePreview.jsx
+++ b/src/components/Dashboard/common/FilePreview.jsx
@ -18,7 +18,7 @@ const FilePreview = ({ file, style = {} }) => {
    const objectUrl = await fetchFileContent(file, false)
    setFileObjectUrl(objectUrl)
    setLoading(false)
-  }, [file._id, fetchFileContent])
+  }, [file, fetchFileContent])

  useEffect(() => {
    if (file?.type && token != null) {
--- a/src/components/Dashboard/common/GCodePreview.jsx
+++ b/src/components/Dashboard/common/GCodePreview.jsx
@ -42,7 +42,14 @@ function GCodePreviewUI(props) {
    return () => {
      window.removeEventListener('resize', resizePreview)
    }
-  }, [endLayer, lastSegmentColor, lineWidth, startLayer, topLayerColor])
+  }, [
+    endLayer,
+    lastSegmentColor,
+    lineWidth,
+    startLayer,
+    topLayerColor,
+    resizePreview
+  ])

  useEffect(() => {
    const loadFromSrc = async () => {
--- a/src/components/Dashboard/common/KeyboardShortcut.jsx
+++ b/src/components/Dashboard/common/KeyboardShortcut.jsx
@ -1,4 +1,4 @@
-import { useEffect, useRef, cloneElement } from 'react'
+import { useEffect, useRef, cloneElement, useCallback } from 'react'
 import PropTypes from 'prop-types'
 import { Popover, Typography } from 'antd'

@ -24,10 +24,9 @@ const { Text } = Typography
 const KeyboardShortcut = ({ shortcut, children, hint, onTrigger }) => {
  const childRef = useRef()
  const shortcutObj = parseShortcut(shortcut)
-  let pressedKeys = new Set()

  // Helper to get the set of keys required for the shortcut
-  function getShortcutKeySet(shortcutObj) {
+  const getShortcutKeySet = useCallback((shortcutObj) => {
    const keys = []
    if (shortcutObj.meta) keys.push('Meta')
    if (shortcutObj.ctrl) keys.push('Control')
@ -38,12 +37,12 @@ const KeyboardShortcut = ({ shortcut, children, hint, onTrigger }) => {
      keys.push('Key' + shortcutObj.key.toUpperCase())
    }
    return new Set(keys)
-  }
+  }, [])

  const shortcutKeySet = getShortcutKeySet(shortcutObj)

  useEffect(() => {
-    pressedKeys = new Set()
+    const pressedKeys = new Set()
    const handleKeyDown = (event) => {
      if (
        event.key === 'Meta' ||
@ -83,7 +82,7 @@ const KeyboardShortcut = ({ shortcut, children, hint, onTrigger }) => {
      window.removeEventListener('keydown', handleKeyDown)
      window.removeEventListener('keyup', handleKeyUp)
    }
-  }, [shortcut, shortcutObj, onTrigger])
+  }, [shortcut, shortcutObj, onTrigger, shortcutKeySet])

  // Clone the child to attach a ref
  const element = cloneElement(children, { ref: childRef })
--- a/src/components/Dashboard/common/NoteItem.jsx
+++ b/src/components/Dashboard/common/NoteItem.jsx
@ -96,12 +96,6 @@ const NoteItem = ({ note }) => {
          'note',
          (noteData) => {
            if (noteData.parent._id == note._id) {
-              console.log(
-                'Note note added to parent:',
-                note._id,
-                'isExpanded:',
-                isExpanded
-              )
              if (isExpanded == true) {
                handleNoteExpand()
              }
--- a/src/components/Dashboard/common/ObjectChildTable.jsx
+++ b/src/components/Dashboard/common/ObjectChildTable.jsx
@ -272,7 +272,7 @@ const ObjectChildTable = ({
    })

    return [summaryRow]
-  }, [properties, rollups, objectData])
+  }, [properties, rollups, objectData, value])

  const rollupColumns = useMemo(() => {
    const propertyColumns = properties.map((property, index) => {
--- a/src/components/Dashboard/common/ObjectDisplay.jsx
+++ b/src/components/Dashboard/common/ObjectDisplay.jsx
@ -91,7 +91,6 @@ const ObjectDisplay = ({
      cancelled = true
    }
  }, [object, fetchFullObjectIfNeeded])
-  console.log(objectData, objectType)
  if (!objectData) {
    return <Text type='secondary'>n/a</Text>
  }
--- a/src/components/Dashboard/common/ObjectForm.jsx
+++ b/src/components/Dashboard/common/ObjectForm.jsx
@ -381,7 +381,6 @@ const ObjectForm = forwardRef(
        ...computedValuesObject,
        _isEditing: isEditingRef.current
      }))
-      console.log('calculatedEntries', computedValuesObject)
      onStateChangeRef.current({
        isEditing: true,
        objectData: {
--- a/src/components/Dashboard/common/ObjectSelect.jsx
+++ b/src/components/Dashboard/common/ObjectSelect.jsx
@ -279,7 +279,6 @@ const ObjectSelect = ({

  // --- loadData for async loading on expand ---
  const loadData = async (node) => {
-    console.log('loading data for node', node)
    // node.property is the property name, node.value is the value key
    if (!node.property) return
    if (type == 'unknown') return
@ -340,28 +339,31 @@ const ObjectSelect = ({
    })
  }

-  const onTreeSelectChange = (value) => {
-    // Mark this as an internal change
-    isInternalChangeRef.current = true
+  const onTreeSelectChange = useCallback(
+    (value) => {
+      // Mark this as an internal change
+      isInternalChangeRef.current = true

-    // value can be a string (single) or array (multiple)
-    if (multiple) {
-      // Multiple selection
-      let selectedObjects = []
-      if (Array.isArray(value)) {
-        selectedObjects = value
-          .map((id) => objectList.find((obj) => obj._id === id))
-          .filter(Boolean)
+      // value can be a string (single) or array (multiple)
+      if (multiple) {
+        // Multiple selection
+        let selectedObjects = []
+        if (Array.isArray(value)) {
+          selectedObjects = value
+            .map((id) => objectList.find((obj) => obj._id === id))
+            .filter(Boolean)
+        }
+        setTreeSelectValue(value)
+        if (rest.onChange) rest.onChange(selectedObjects)
+      } else {
+        // Single selection
+        const selectedObject = objectList.find((obj) => obj._id === value)
+        setTreeSelectValue(value)
+        if (rest.onChange) rest.onChange(selectedObject)
      }
-      setTreeSelectValue(value)
-      if (rest.onChange) rest.onChange(selectedObjects)
-    } else {
-      // Single selection
-      const selectedObject = objectList.find((obj) => obj._id === value)
-      setTreeSelectValue(value)
-      if (rest.onChange) rest.onChange(selectedObject)
-    }
-  }
+    },
+    [multiple, objectList, rest]
+  )

  // Update treeData when objectPropertiesTree changes
  useEffect(() => {
@ -385,11 +387,9 @@ const ObjectSelect = ({
        getValueIdentity(valueRef.current) !== getValueIdentity(value) &&
        type != 'unknown'
      ) {
-        // console.log('fetching full object', value)
        valueRef.current = value
        // Check if value is a minimal object and fetch full object if needed
        const fullValue = await fetchFullObjectIfNeeded(value)
-        // console.log('fullValue', fullValue)
        // Build a new filter from value's properties that are in the properties list
        const valueFilter = { ...filter }
        const pathKeys = []
@ -427,7 +427,6 @@ const ObjectSelect = ({
        setExpandedKeys(pathKeys)
        // Fetch with the new filter
        handleFetchObjectsProperties(valueFilter)
-        // console.log('setting treeSelectValue', valueRef.current._id)
        setTreeSelectValue(valueRef.current._id)
        setInitialized(true)
        return
@ -469,10 +468,6 @@ const ObjectSelect = ({

  const prevValuesRef = useRef({ type, masterFilter })

-  useEffect(() => {
-    // console.log('treeSelectValue', treeSelectValue)
-  }, [treeSelectValue])
-
  useEffect(() => {
    const prevValues = prevValuesRef.current

@ -494,7 +489,7 @@ const ObjectSelect = ({
      setError(false)
      prevValuesRef.current = { type, masterFilter }
    }
-  }, [type, masterFilter])
+  }, [type, masterFilter, onTreeSelectChange])

  useEffect(() => {
    // Check if value has actually changed
@ -519,7 +514,7 @@ const ObjectSelect = ({
      prevValueRef.current = value
      prevValueIdentityRef.current = currentValueIdentity
    }
-  }, [value, getValueIdentity])
+  }, [value, getValueIdentity, type, masterFilter])

  const placeholder = useMemo(
    () =>
--- a/src/components/Dashboard/common/ObjectTable.jsx
+++ b/src/components/Dashboard/common/ObjectTable.jsx
@ -236,7 +236,6 @@ const ObjectTable = forwardRef(
            order: sorter.order
          }
        }
-        console.log('Fetching data...')
        try {
          const result = await fetchObjects(type, {
            page: pageNum,
@ -252,7 +251,6 @@ const ObjectTable = forwardRef(
            const existingPageIndex = prev.findIndex(
              (p) => p.pageNum === pageNum
            )
-            logger.debug(prev.map((p) => p.pageNum))
            if (existingPageIndex !== -1) {
              // Update existing page
              const newPages = [...prev]
@ -370,7 +368,6 @@ const ObjectTable = forwardRef(
      setIsEditing(true)
      tableData.forEach((item) => {
        if (!item.isSkeleton) {
-          console.log('Locking object:', item)
          lockObject(item._id, type)
        }
      })
@ -403,7 +400,7 @@ const ObjectTable = forwardRef(
          }
        })
      } catch (err) {
-        console.error(err)
+        logger.error('Error updating objects:', err)
      } finally {
        setEditLoading(false)
      }
@ -426,8 +423,6 @@ const ObjectTable = forwardRef(
        })
      )

-      console.log('updatedData', updatedData)
-
      if (rowFormsRef.current[id]) {
        rowFormsRef.current[id].setFieldsValue(updatedData)
      }
@ -464,9 +459,6 @@ const ObjectTable = forwardRef(
              updateEventHandlerRef.current(itemId, updateData)
            }
          )
-          console.log('unsubscribe', unsubscribe)
-          console.log('subscribedIdsRef', subscribedIdsRef.current)
-          console.log('unsubscribesRef', unsubscribesRef.current)
          subscribedIdsRef.current.push(itemId)
          if (unsubscribe) {
            unsubscribesRef.current.push(unsubscribe)
--- a/src/components/Dashboard/common/PrinterMiscPanel.jsx
+++ b/src/components/Dashboard/common/PrinterMiscPanel.jsx
@ -16,7 +16,6 @@ const PrinterMiscPanel = ({ id, showControls = true }) => {
      target: 0
    },
    lcdBacklight: {
-      // eslint-disable-line camelcase
      brightness: 0
    },
    beeper: {
--- a/src/components/Dashboard/common/PrinterPositionPanel.jsx
+++ b/src/components/Dashboard/common/PrinterPositionPanel.jsx
@ -38,14 +38,14 @@ const PrinterPositionPanel = ({
  const { subscribeToObjectEvent, connected, sendObjectAction } =
    useContext(ApiServerContext)
  const [positionData, setPositionData] = useState({
-    speedFactor: 1.0, // eslint-disable-line
+    speedFactor: 1.0,
    speed: 100,
-    extrudeFactor: 1.0, // eslint-disable-line
-    absoluteCoordinates: true, // eslint-disable-line
-    absoluteExtrude: false, // eslint-disable-line
-    homingOrigin: [0.0, 0.0, 0.0, 0.0], // eslint-disable-line
+    extrudeFactor: 1.0,
+    absoluteCoordinates: true,
+    absoluteExtrude: false,
+    homingOrigin: [0.0, 0.0, 0.0, 0.0],
    toolheadPosition: [0.0, 0.0, 0.0, 0.0],
-    gcodePosition: [0.0, 0.0, 0.0, 0.0], // eslint-disable-line
+    gcodePosition: [0.0, 0.0, 0.0, 0.0],
    livePosition: [0.0, 0.0, 0.0, 0.0],
    maxVelocity: 1000,
    maxAcceleration: 1000,
@ -70,7 +70,7 @@ const PrinterPositionPanel = ({
        if (motionEventUnsubscribe) motionEventUnsubscribe()
      }
    }
-  }, [id, connected])
+  }, [id, connected, subscribeToObjectEvent])
  const [speedFactor, setSpeedFactor] = useState(positionData.speedFactor)
  const [extrudeFactor, setExtrudeFactor] = useState(positionData.extrudeFactor)
  const [maxVelocity, setMaxVelocity] = useState(positionData.maxVelocity)
--- a/src/components/Dashboard/common/PrinterTemperaturePanel.jsx
+++ b/src/components/Dashboard/common/PrinterTemperaturePanel.jsx
@ -87,7 +87,7 @@ const PrinterTemperaturePanel = ({
        if (temperatureEventUnsubscribe) temperatureEventUnsubscribe()
      }
    }
-  }, [id, connected])
+  }, [id, connected, subscribeToObjectEvent])

  const [extruderTarget, setExtruderTarget] = useState(0)
  const [bedTarget, setBedTarget] = useState(0)
--- a/src/components/Dashboard/common/SpotlightTooltip.jsx
+++ b/src/components/Dashboard/common/SpotlightTooltip.jsx
@ -40,7 +40,6 @@ const SpotlightTooltip = ({ query, type }) => {
  useEffect(() => {
    if (token != null && initialized == false) {
      fetchSpotlight()
-      console.log('Fetching spotlight...')
      setInitialized(true)
    }
  }, [token, fetchSpotlight, initialized])
--- a/src/components/Dashboard/common/TemplatePreview.jsx
+++ b/src/components/Dashboard/common/TemplatePreview.jsx
@ -44,22 +44,25 @@ const TemplatePreview = ({
    }
  }

-  const reloadPreviewPDF = (content, testObject = {}) => {
-    setReloadLoading(true)
-    fetchTemplatePDF(documentTemplate._id, content, testObject, (result) => {
-      setReloadLoading(false)
-      if (result?.error) {
-        // Handle error through parent component
-        onPreviewMessage(result.error, true)
-      } else {
-        const pdfBlob = new Blob([result.pdf], { type: 'application/pdf' })
-        const pdfUrl = URL.createObjectURL(pdfBlob)
+  const reloadPreviewPDF = useCallback(
+    (content, testObject = {}) => {
+      setReloadLoading(true)
+      fetchTemplatePDF(documentTemplate._id, content, testObject, (result) => {
+        setReloadLoading(false)
+        if (result?.error) {
+          // Handle error through parent component
+          onPreviewMessage(result.error, true)
+        } else {
+          const pdfBlob = new Blob([result.pdf], { type: 'application/pdf' })
+          const pdfUrl = URL.createObjectURL(pdfBlob)

-        setPDFBlob(pdfUrl)
-        onPreviewMessage('No issues found.', false)
-      }
-    })
-  }
+          setPDFBlob(pdfUrl)
+          onPreviewMessage('No issues found.', false)
+        }
+      })
+    },
+    [documentTemplate?._id, fetchTemplatePDF, onPreviewMessage]
+  )

  const reloadPreview = useCallback(
    (content, testObject = {}, scale = 1) => {
@ -81,7 +84,7 @@ const TemplatePreview = ({
        }
      )
    },
-    [fetchTemplatePreview, objectData?._id, onPreviewMessage]
+    [fetchTemplatePreview, onPreviewMessage, documentTemplate?._id]
  )

  // Move useEffect to component level and use state to track objectData changes
@ -93,7 +96,14 @@ const TemplatePreview = ({
        reloadPreviewPDF(documentTemplate.content, objectData)
      }
    }
-  }, [objectData, documentTemplate, previewScale, previewType])
+  }, [
+    objectData,
+    documentTemplate,
+    previewScale,
+    previewType,
+    reloadPreview,
+    reloadPreviewPDF
+  ])

  return (
    <Flex vertical gap={'middle'} style={{ height: '100%' }}>
--- a/src/components/Dashboard/common/ThreeDPreview.jsx
+++ b/src/components/Dashboard/common/ThreeDPreview.jsx
@ -10,10 +10,7 @@ function ThreeDPreview(props) {
    width = 500,
    height = 500,
    style = {},
-    backgroundColor = '#ffffff',
-    showGrid = true,
-    showAxes = true,
-    enableControls = true
+    backgroundColor = '#ffffff'
  } = props
  const containerRef = useRef(null)
  const viewer = useRef(null)
@ -34,6 +31,9 @@ function ThreeDPreview(props) {
  }, [viewer, width, height])

  useEffect(() => {
+    // Variable to track the viewer instance created in this effect
+    let currentViewer = null
+
    const initializeViewer = async () => {
      if (!containerRef.current) return

@ -42,7 +42,10 @@ function ThreeDPreview(props) {
        setError(null)

        // Clear any existing viewer
-        if (viewer) {
+        if (viewer.current) {
+          if (viewer.current.dispose) {
+            viewer.current.dispose()
+          }
          containerRef.current.innerHTML = ''
        }

@ -71,6 +74,10 @@ function ThreeDPreview(props) {
          environmentSettings: new OV.EnvironmentSettings([], false)
        })

+        // Store the viewer instance in the ref and local variable
+        viewer.current = newViewer
+        currentViewer = newViewer
+
        try {
          setIsLoading(true)
          setError(null)
@ -105,11 +112,11 @@ function ThreeDPreview(props) {

    return () => {
      window.removeEventListener('resize', resizeViewer)
-      if (viewer.current && viewer.current.dispose) {
-        viewer.current.dispose()
+      if (currentViewer && currentViewer.dispose) {
+        currentViewer.dispose()
      }
    }
-  }, [width, height, backgroundColor, showGrid, showAxes, enableControls, src])
+  }, [width, height, src, extension, resizeViewer])

  const containerStyle = {
    width: width + 'px',
--- a/src/components/Dashboard/context/ApiServerContext.jsx
+++ b/src/components/Dashboard/context/ApiServerContext.jsx
@ -8,7 +8,7 @@ import {
  useCallback
 } from 'react'
 import io from 'socket.io-client'
-import { message, notification, Modal, Space, Button } from 'antd'
+import { message, Modal, Space, Button } from 'antd'
 import PropTypes from 'prop-types'
 import { AuthContext } from './AuthContext'

@ -30,7 +30,6 @@ const ApiServerProvider = ({ children }) => {
  const [connecting, setConnecting] = useState(false)
  const [error, setError] = useState(null)
  const [messageApi, contextHolder] = message.useMessage()
-  const [notificationApi] = notification.useNotification()
  const [fetchLoading, setFetchLoading] = useState(false)
  const [showErrorModal, setShowErrorModal] = useState(false)
  const [errorModalContent, setErrorModalContent] = useState('')
@ -125,7 +124,7 @@ const ApiServerProvider = ({ children }) => {

      socketRef.current = newSocket
    }
-  }, [token, authenticated, messageApi, notificationApi, handleLockUpdate])
+  }, [token, authenticated, messageApi, handleLockUpdate, clearSubscriptions])

  useEffect(() => {
    if (token && authenticated == true) {
@ -254,7 +253,6 @@ const ApiServerProvider = ({ children }) => {

    const callbacksRefKey = `modelStats:${objectType}`
    logger.debug('Notifying model stats update:', data)
-    console.log('handleModelStats', data)
    if (objectType && subscribedCallbacksRef.current.has(callbacksRefKey)) {
      const callbacks = subscribedCallbacksRef.current.get(callbacksRefKey)
      logger.debug(
@ -539,7 +537,7 @@ const ApiServerProvider = ({ children }) => {
        return () => offObjectEventEvent(id, objectType, eventType, callback)
      }
    },
-    [offObjectUpdatesEvent]
+    [offObjectEventEvent]
  )

  const offModelStats = useCallback((objectType, callback) => {
--- a/src/components/Dashboard/context/AuthContext.jsx
+++ b/src/components/Dashboard/context/AuthContext.jsx
@ -217,7 +217,7 @@ const AuthProvider = ({ children }) => {
            setExpiresAt(newExpiresAt)
            setUserProfile(newUser)
            setAuthenticated(true)
-            console.log('Auth state synchronized from another tab')
+            logger.debug('Auth state synchronized from another tab')
          }
        } else {
          // Cookies are invalid, clear state
@ -226,7 +226,7 @@ const AuthProvider = ({ children }) => {
          setUserProfile(null)
          setAuthenticated(false)
          setShowUnauthorizedModal(true)
-          console.log(
+          logger.debug(
            'Auth state cleared due to invalid cookies from another tab'
          )
        }
@ -256,11 +256,11 @@ const AuthProvider = ({ children }) => {
      messageApi.info('Logging in with tombutcher.work')
      const loginUrl = `${config.backendUrl}/auth/${redirectType}/login?redirect_uri=${encodeURIComponent(redirectUri)}`
      if (isElectron) {
-        console.log('Opening external url...')
+        logger.debug('Opening external url...')
        openExternalUrl(loginUrl)
        setLoading(true)
      } else {
-        console.log('Redirecting...')
+        logger.debug('Redirecting...')
        window.location.href = loginUrl
      }
    },
@ -338,12 +338,12 @@ const AuthProvider = ({ children }) => {
      }
    },
    [
-      isElectron,
      navigate,
      location.search,
      location.pathname,
      messageApi,
-      persistSession
+      persistSession,
+      redirectType
    ]
  )

--- a/src/components/Dashboard/context/ElectronContext.jsx
+++ b/src/components/Dashboard/context/ElectronContext.jsx
@ -7,6 +7,7 @@ const electron = window.require ? window.require('electron') : null
 const ipcRenderer = electron ? electron.ipcRenderer : null

 // Utility to check if running in Electron
+// eslint-disable-next-line react-refresh/only-export-components
 export function isElectron() {
  // Renderer process

@ -87,7 +88,6 @@ const ElectronProvider = ({ children }) => {

    // Listen for navigate
    const navigateHandler = (event, url) => {
-      console.log('Navigating to:', url)
      navigate(url)
    }
    ipcRenderer.on('navigate', navigateHandler)
@ -96,7 +96,7 @@ const ElectronProvider = ({ children }) => {
      ipcRenderer.removeListener('navigate', navigateHandler)
      ipcRenderer.removeListener('window-state', windowStateHandler)
    }
-  }, [])
+  }, [navigate])

  // Window control handler
  const handleWindowControl = (action) => {
--- a/src/components/Dashboard/context/MessageContext.jsx
+++ b/src/components/Dashboard/context/MessageContext.jsx
@ -47,6 +47,7 @@ MessageProvider.propTypes = {
  children: PropTypes.node.isRequired
 }

+// eslint-disable-next-line react-refresh/only-export-components
 export const useMessageContext = () => {
  const context = useContext(MessageContext)
  if (!context) {
--- a/src/components/Dashboard/context/SpotlightContext.jsx
+++ b/src/components/Dashboard/context/SpotlightContext.jsx
@ -17,7 +17,8 @@ import {
  useState,
  useRef,
  createElement,
-  useContext
+  useContext,
+  useCallback
 } from 'react'
 import axios from 'axios'
 import { LoadingOutlined } from '@ant-design/icons'
@ -608,7 +609,7 @@ const ElectronSpotlightContentPage = () => {
  const { resizeSpotlightWindow, isElectron } = useContext(ElectronContext)

  // Function to measure and resize window
-  const updateWindowHeight = () => {
+  const updateWindowHeight = useCallback(() => {
    if (!cardRef.current || !isElectron || !resizeSpotlightWindow) return

    // Clear any pending resize
@ -639,7 +640,7 @@ const ElectronSpotlightContentPage = () => {
        console.warn('Failed to update spotlight window height:', error)
      }
    }, 100) // 100ms debounce
-  }
+  }, [isElectron, resizeSpotlightWindow])

  // Use ResizeObserver to watch for content changes
  useEffect(() => {
@ -666,7 +667,7 @@ const ElectronSpotlightContentPage = () => {
      }
      clearTimeout(initialTimeout)
    }
-  }, [isElectron, resizeSpotlightWindow])
+  }, [isElectron, resizeSpotlightWindow, updateWindowHeight])

  return (
    <div
--- a/src/components/Dashboard/context/ThemeContext.jsx
+++ b/src/components/Dashboard/context/ThemeContext.jsx
@ -135,6 +135,7 @@ ThemeProvider.propTypes = {
  children: PropTypes.node.isRequired
 }

+// eslint-disable-next-line react-refresh/only-export-components
 export const useThemeContext = () => {
  const context = useContext(ThemeContext)
  if (!context) {
--- a/src/database/models/Courier.js
+++ b/src/database/models/Courier.js
@ -55,7 +55,6 @@ export const Courier = {
      url: (_id) =>
        `/dashboard/management/couriers/info?courierId=${_id}&action=cancelEdit`,
      visible: (objectData) => {
-        console.log(objectData?._isEditing)
        return objectData?._isEditing && objectData?._isEditing == true
      }
    },
--- a/src/database/models/CourierService.js
+++ b/src/database/models/CourierService.js
@ -56,7 +56,6 @@ export const CourierService = {
      url: (_id) =>
        `/dashboard/management/courierservices/info?courierServiceId=${_id}&action=cancelEdit`,
      visible: (objectData) => {
-        console.log(objectData?._isEditing)
        return objectData?._isEditing && objectData?._isEditing == true
      }
    },
@ -70,14 +69,7 @@ export const CourierService = {
        `/dashboard/management/courierservices/info?courierServiceId=${_id}&action=delete`
    }
  ],
-  columns: [
-    'name',
-    '_id',
-    'courier',
-    'tracked',
-    'deliveryTime',
-    'active'
-  ],
+  columns: ['name', '_id', 'courier', 'tracked', 'deliveryTime', 'active'],
  filters: ['name', '_id', 'courier', 'active', 'deliveryTime', 'tracked'],
  sorters: [
    'name',
--- a/src/database/models/DocumentJob.js
+++ b/src/database/models/DocumentJob.js
@ -57,7 +57,6 @@ export const DocumentJob = {
      url: (_id) =>
        `/dashboard/management/documentjobs/info?documentJobId=${_id}&action=cancelEdit`,
      visible: (objectData) => {
-        console.log(objectData?._isEditing)
        return objectData?._isEditing && objectData?._isEditing == true
      }
    }
--- a/src/database/models/DocumentPrinter.js
+++ b/src/database/models/DocumentPrinter.js
@ -56,20 +56,11 @@ export const DocumentPrinter = {
      url: (_id) =>
        `/dashboard/management/documentprinters/info?documentPrinterId=${_id}&action=cancelEdit`,
      visible: (objectData) => {
-        console.log(objectData?._isEditing)
        return objectData?._isEditing && objectData?._isEditing == true
      }
    }
  ],
-  columns: [
-    'name',
-    '_id',
-    'state',
-    'host',
-    'tags',
-    'connectedAt',
-    'updatedAt'
-  ],
+  columns: ['name', '_id', 'state', 'host', 'tags', 'connectedAt', 'updatedAt'],
  filters: ['name', '_id'],
  sorters: ['name', 'documentSize', 'connectedAt', 'updatedAt'],
  properties: [
--- a/src/database/models/DocumentSize.js
+++ b/src/database/models/DocumentSize.js
@ -56,7 +56,6 @@ export const DocumentSize = {
      url: (_id) =>
        `/dashboard/management/documentsizes/info?documentSizeId=${_id}&action=cancelEdit`,
      visible: (objectData) => {
-        console.log(objectData?._isEditing)
        return objectData?._isEditing && objectData?._isEditing == true
      }
    }
--- a/src/database/models/DocumentTemplate.js
+++ b/src/database/models/DocumentTemplate.js
@ -65,7 +65,6 @@ export const DocumentTemplate = {
      url: (_id) =>
        `/dashboard/management/documenttemplates/info?documentTemplateId=${_id}&action=cancelEdit`,
      visible: (objectData) => {
-        console.log(objectData?._isEditing)
        return objectData?._isEditing && objectData?._isEditing == true
      }
    }
--- a/src/database/models/Filament.js
+++ b/src/database/models/Filament.js
@ -54,7 +54,6 @@ export const Filament = {
      url: (_id) =>
        `/dashboard/management/filaments/info?filamentId=${_id}&action=cancelEdit`,
      visible: (objectData) => {
-        console.log(objectData?._isEditing)
        return objectData?._isEditing && objectData?._isEditing == true
      }
    }
--- a/src/database/models/File.js
+++ b/src/database/models/File.js
@ -33,7 +33,8 @@ export const File = {
      label: 'Edit',
      row: true,
      icon: EditIcon,
-      url: (_id) => `/dashboard/management/files/info?fileId=${_id}&action=edit`,
+      url: (_id) =>
+        `/dashboard/management/files/info?fileId=${_id}&action=edit`,
      visible: (objectData) => {
        return !(objectData?._isEditing && objectData?._isEditing == true)
      }
@ -55,7 +56,6 @@ export const File = {
      url: (_id) =>
        `/dashboard/management/files/info?fileId=${_id}&action=cancelEdit`,
      visible: (objectData) => {
-        console.log(objectData?._isEditing)
        return objectData?._isEditing && objectData?._isEditing == true
      }
    },
--- a/src/database/models/GCodeFile.js
+++ b/src/database/models/GCodeFile.js
@ -63,7 +63,6 @@ export const GCodeFile = {
      url: (_id) =>
        `/dashboard/production/gcodefiles/info?gcodeFileId=${_id}&action=cancelEdit`,
      visible: (objectData) => {
-        console.log(objectData?._isEditing)
        return objectData?._isEditing && objectData?._isEditing == true
      }
    }
--- a/src/database/models/Host.js
+++ b/src/database/models/Host.js
@ -63,7 +63,6 @@ export const Host = {
      url: (_id) =>
        `/dashboard/management/hosts/info?hostId=${_id}&action=cancelEdit`,
      visible: (objectData) => {
-        console.log(objectData?._isEditing)
        return objectData?._isEditing && objectData?._isEditing == true
      }
    }
--- a/src/database/models/Job.js
+++ b/src/database/models/Job.js
@ -27,7 +27,6 @@ export const Job = {
      url: (_id) =>
        `/dashboard/production/jobs/info?jobId=${_id}&action=deploy`,
      disabled: (objectData) => {
-        console.log('Should be disabled', objectData?.state?.type != 'draft')
        return objectData?.state?.type != 'draft'
      }
    },
--- a/src/database/models/NoteType.js
+++ b/src/database/models/NoteType.js
@ -54,7 +54,6 @@ export const NoteType = {
      url: (_id) =>
        `/dashboard/management/notetypes/info?noteTypeId=${_id}&action=cancelEdit`,
      visible: (objectData) => {
-        console.log(objectData?._isEditing)
        return objectData?._isEditing && objectData?._isEditing == true
      }
    }
--- a/src/database/models/Part.js
+++ b/src/database/models/Part.js
@ -53,7 +53,6 @@ export const Part = {
      url: (_id) =>
        `/dashboard/management/parts/info?partId=${_id}&action=cancelEdit`,
      visible: (objectData) => {
-        console.log(objectData?._isEditing)
        return objectData?._isEditing && objectData?._isEditing == true
      }
    }
@ -98,7 +97,6 @@ export const Part = {
      objectType: 'vendor',
      showHyperlink: true,
      value: (objectData) => {
-        console.log(objectData?.vendor, objectData?.product?.vendor)
        if (!objectData?.vendor && objectData?.product?.vendor) {
          return objectData?.product?.vendor
        } else {
--- a/src/database/models/Printer.js
+++ b/src/database/models/Printer.js
@ -68,7 +68,6 @@ export const Printer = {
      url: (_id) =>
        `/dashboard/production/printers/info?printerId=${_id}&action=cancelEdit`,
      visible: (objectData) => {
-        console.log(objectData?._isEditing)
        return objectData?._isEditing && objectData?._isEditing == true
      }
    },
@ -127,7 +126,6 @@ export const Printer = {
          label: 'Start Queue',
          icon: PlayCircleIcon,
          disabled: (objectData) => {
-            console.log(objectData?.queue?.length)
            return (
              objectData?.state?.type == 'error' ||
              objectData?.state?.type == 'printing' ||
--- a/src/database/models/Product.js
+++ b/src/database/models/Product.js
@ -54,20 +54,11 @@ export const Product = {
      url: (_id) =>
        `/dashboard/management/products/info?productId=${_id}&action=cancelEdit`,
      visible: (objectData) => {
-        console.log(objectData?._isEditing)
        return objectData?._isEditing && objectData?._isEditing == true
      }
    }
  ],
-  columns: [
-    '_id',
-    'name',
-    'tags',
-    'vendor',
-    'price',
-    'createdAt',
-    'updatedAt'
-  ],
+  columns: ['_id', 'name', 'tags', 'vendor', 'price', 'createdAt', 'updatedAt'],
  filters: ['_id', 'name', 'type', 'color', 'cost', 'vendor'],
  sorters: ['name', 'createdAt', 'type', 'vendor', 'cost', 'updatedAt'],
  properties: [
--- a/src/database/models/Vendor.js
+++ b/src/database/models/Vendor.js
@ -55,7 +55,6 @@ export const Vendor = {
      url: (_id) =>
        `/dashboard/management/vendors/info?vendorId=${_id}&action=cancelEdit`,
      visible: (objectData) => {
-        console.log(objectData?._isEditing)
        return objectData?._isEditing && objectData?._isEditing == true
      }
    },
--- a/src/utils/cookies.js
+++ b/src/utils/cookies.js
@ -21,8 +21,6 @@ export const setCookie = (name, value, options = {}) => {

    const cookieOptions = { ...COOKIE_OPTIONS, ...options }

-    console.log('VALUE', value)
-
    let cookieString = `${name}=${encodeURIComponent(value)}`

    if (cookieOptions.maxAge) {
@ -74,7 +72,6 @@ export const getCookie = (name) => {

    for (let i = 0; i < cookies.length; i++) {
      let cookie = cookies[i]
-      console.log(cookie)
      while (cookie.charAt(0) === ' ') {
        cookie = cookie.substring(1, cookie.length)
      }
@ -123,7 +120,7 @@ export const areCookiesEnabled = () => {
    removeCookie('test')
    return enabled
  } catch (e) {
-    console.log(e)
+    console.error('Error checking if cookies are enabled:', e)
    return false
  }
 }
--- a/vite.config.js
+++ b/vite.config.js
@ -5,7 +5,7 @@ import svgr from 'vite-plugin-svgr'
 import svgo from 'vite-plugin-svgo'

 export default defineConfig({
-  base: './',
+  base: '/',
  plugins: [react(), svgo(), svgr(), eslintPlugin()],
  build: {
    outDir: 'build',
Author	SHA1	Message	Date
Tom Butcher	7e1ec9f3ae	Removed development logging.	2025-12-27 15:19:09 +00:00
Tom Butcher	f6986a45cd	Fixed all warnings.	2025-12-27 15:06:02 +00:00
Tom Butcher	e6ebc16009	Fixed base.	2025-12-27 14:32:00 +00:00