fix(target): add back target auto-refresh option (#577)

* fix(target): add back target auto-refresh option * chore(hooks): remove unused hook deps
cryostatio · Oct 27, 2022 · 8dd429d · 8dd429d
1 parent 1155826
commit 8dd429d
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/src/app/TargetSelect/TargetSelect.tsx b/src/app/TargetSelect/TargetSelect.tsx
@@ -141,6 +141,22 @@ export const TargetSelect: React.FunctionComponent<TargetSelectProps> = (props)
     addSubscription(context.target.target().subscribe(setSelected));
   }, [addSubscription, context.target, setSelected]);
 
+  const refreshTargetList = React.useCallback(() => {
+    setLoading(true);
+    addSubscription(context.targets.queryForTargets().subscribe(() => setLoading(false)));
+  }, [addSubscription, context.targets, setLoading]);
+
+  React.useEffect(() => {
+    if (!context.settings.autoRefreshEnabled()) {
+      return;
+    }
+    const id = window.setInterval(
+      () => refreshTargetList(),
+      context.settings.autoRefreshPeriod() * context.settings.autoRefreshUnits()
+    );
+    return () => window.clearInterval(id);
+  }, [context.settings, refreshTargetList]);
+
   const showCreateTargetModal = React.useCallback(() => {
     setModalOpen(true);
   }, [setModalOpen]);