fix: skip Apollo query when folders state exists (#2826)

webiny · Dec 6, 2022 · 6a9f83c · 6a9f83c
1 parent 78f581c
commit 6a9f83c
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/packages/app-folders/src/hooks/useFolders.ts b/packages/app-folders/src/hooks/useFolders.ts
@@ -15,9 +15,15 @@ export const useFolders = (type: string) => {
         /**
          * On first mount, call `listFolders`, which will either issue a network request, or load folders from cache.
          * We don't need to store the result of it to any local state; that is managed by the context provider.
+         *
+         * IMPORTANT: we check if the folders[type] array exists: the hook can be used from multiple components and
+         * fetch the outdated list from Apollo Cache. Since the state is managed locally, we fetch the folders only
+         * at the first mount.
          */
-        listFolders(type);
-    }, []);
+        if (!folders[type]) {
+            listFolders(type);
+        }
+    }, [type]);
 
     return useMemo(
         () => ({