Fix: fix fe code to not make requests when validating cache (#720)

Signed-off-by: Daishan Peng <[email protected]>
otto8-ai · Dec 2, 2024 · 24cf54d · 24cf54d
1 parent e149f0c
commit 24cf54d
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 8 deletions.
diff --git a/ui/admin/app/components/knowledge/AgentKnowledgePanel.tsx b/ui/admin/app/components/knowledge/AgentKnowledgePanel.tsx
@@ -97,7 +97,7 @@ export default function AgentKnowledgePanel({
             ),
         {
             revalidateOnFocus: false,
-            refreshInterval: blockPollingLocalFiles ? undefined : 1000,
+            refreshInterval: blockPollingLocalFiles ? undefined : 5000,
         }
     );
     const localFiles = useMemo(
@@ -110,7 +110,7 @@ export default function AgentKnowledgePanel({
         ({ agentId }) => KnowledgeService.getKnowledgeSourcesForAgent(agentId),
         {
             revalidateOnFocus: false,
-            refreshInterval: blockPollingSources ? undefined : 1000,
+            refreshInterval: blockPollingSources ? undefined : 5000,
         }
     );
     const knowledgeSources = useMemo(

diff --git a/ui/admin/app/components/knowledge/KnowledgeSourceDetail.tsx b/ui/admin/app/components/knowledge/KnowledgeSourceDetail.tsx
@@ -97,7 +97,7 @@ const KnowledgeSourceDetail: FC<KnowledgeSourceDetailProps> = ({
             ),
         {
             revalidateOnFocus: false,
-            refreshInterval: blockPollingFiles ? undefined : 1000,
+            refreshInterval: blockPollingFiles ? undefined : 5000,
         }
     );
 
@@ -156,7 +156,7 @@ const KnowledgeSourceDetail: FC<KnowledgeSourceDetailProps> = ({
         if (knowledgeSource.state === KnowledgeSourceStatus.Synced) {
             getFiles.mutate();
         }
-    }, [knowledgeSource, getFiles]);
+    }, [knowledgeSource]);
 
     const onSourceUpdate = async (syncSchedule: string) => {
         const updatedSource = await KnowledgeService.updateKnowledgeSource(
@@ -176,8 +176,9 @@ const KnowledgeSourceDetail: FC<KnowledgeSourceDetailProps> = ({
             file.id,
             approved
         );
-        getFiles.mutate((files) =>
-            files?.map((f) => (f.id === file.id ? updatedFile : f))
+        getFiles.mutate(
+            (files) => files?.map((f) => (f.id === file.id ? updatedFile : f)),
+            false
         );
     };
 
@@ -203,8 +204,9 @@ const KnowledgeSourceDetail: FC<KnowledgeSourceDetailProps> = ({
             file.id,
             knowledgeSource.id
         );
-        getFiles.mutate((files) =>
-            files?.map((f) => (f.id === file.id ? updatedFile : f))
+        getFiles.mutate(
+            (files) => files?.map((f) => (f.id === file.id ? updatedFile : f)),
+            false
         );
     };