fix: electronStore onDidChange error

kubeshop · Mar 25, 2022 · d273a31 · d273a31
1 parent ff8456b
commit d273a31
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 8 deletions.
diff --git a/src/App.tsx b/src/App.tsx
@@ -186,11 +186,13 @@ const App = () => {
   useEffect(() => {
     globalElectronStoreChanges.forEach(globalElectronStoreChange => {
       electronStore.onDidChange(globalElectronStoreChange.keyName, (newData: any, oldData: any) => {
+        if (!newData || !oldData) {
+          return;
+        }
         const {shouldTriggerAcrossWindows, eventData} = globalElectronStoreChange.action(newData, oldData);
         if (!shouldTriggerAcrossWindows || !eventData) {
           return;
         }
-
         ipcRenderer.send('global-electron-store-update', eventData);
       });
     });

diff --git a/src/utils/global-electron-store.ts b/src/utils/global-electron-store.ts
@@ -28,13 +28,12 @@ const projectNameChange: ElectronStoreChangePropagate<Project[], ProjectNameChan
   action: (newProjects, oldProjects) => {
     // means that a new project has been created and we should not update that
     if (newProjects.length > oldProjects.length) {
-      return { shouldTriggerAcrossWindows: false };
+      return {shouldTriggerAcrossWindows: false};
     }
 
-    const triggeredForProject = newProjects
-      .find((project, index) => project.name !== oldProjects[index].name);
+    const triggeredForProject = newProjects.find((project, index) => project.name !== oldProjects[index].name);
     if (!triggeredForProject) {
-      return { shouldTriggerAcrossWindows: false };
+      return {shouldTriggerAcrossWindows: false};
     }
 
     return {
@@ -49,6 +48,4 @@ const projectNameChange: ElectronStoreChangePropagate<Project[], ProjectNameChan
   },
 };
 
-export const globalElectronStoreChanges: ElectronStoreChangePropagate<any, any>[] = [
-  projectNameChange,
-];
+export const globalElectronStoreChanges: ElectronStoreChangePropagate<any, any>[] = [projectNameChange];