Merge pull request #88 from launchrctl/linter

Linter fix
launchrctl · Sep 12, 2024 · cf079bd · cf079bd
2 parents b536345 + ad34ee5
commit cf079bd
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 18 deletions.
diff --git a/client/src/components/GlobalKBarProvider.tsx b/client/src/components/GlobalKBarProvider.tsx
@@ -19,21 +19,29 @@ export const GlobalKBarProvider: FC<Props> = ({ children }) => {
   const dispatch = useActionDispatch()
   const navigate = useNavigate()
 
-  if (!isError && !isLoading) {
-    const kbar: KBarProviderProps['actions'] = []
-    data?.data?.forEach((action) => {
-      kbar.push({
-        id: action.id,
-        name: `${action.title} (${action.id})`,
-        perform: () => {
-          navigate('/flow')
-          dispatch?.({
-            type: 'set-active-action',
-            id: action.id,
-          })
-        },
-      })
-    })
-    return <KBarProvider actions={kbar}>{children}</KBarProvider>
+  if (isError) {
+    return null
+  }
+
+  if (isLoading) {
+    return null
   }
+
+  const kbar: KBarProviderProps['actions'] = []
+
+  data?.data?.forEach((action) => {
+    kbar.push({
+      id: action.id,
+      name: `${action.title} (${action.id})`,
+      perform: () => {
+        navigate('/flow')
+        dispatch?.({
+          type: 'set-active-action',
+          id: action.id,
+        })
+      },
+    })
+  })
+
+  return <KBarProvider actions={kbar}>{children}</KBarProvider>
 }
diff --git a/client/src/components/SidebarTree.tsx b/client/src/components/SidebarTree.tsx
@@ -347,11 +347,15 @@ export const SidebarTree: FC<{
       })()
       setExpandedItems(expandItems.reverse())
       if (selectedAction) {
-        const prevSelectedAction = apiRef.current?.getItem(selectedAction)
+        const prevSelectedAction = (
+          apiRef.current as { getItem: (id: string) => any }
+        )?.getItem(selectedAction)
         prevSelectedAction.selected = false
         setSelectedAction('')
       }
-      const curSelectedAction = apiRef.current?.getItem(nodeId)
+      const curSelectedAction = (
+        apiRef.current as { getItem: (id: string) => any }
+      )?.getItem(nodeId)
       if (curSelectedAction?.selected) {
         curSelectedAction.selected = nodeId
       }