Merge pull request #1237 from yaacov/set-loading-when-changing-provider

🐞 Set loading when changing provider
kubev2v · Jun 25, 2024 · c92d03f · c92d03f
2 parents 4750252 + 562327a
commit c92d03f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/...console-plugin/src/modules/Plans/views/create/components/ProvidersVirtualMachinesList.tsx b/...console-plugin/src/modules/Plans/views/create/components/ProvidersVirtualMachinesList.tsx
@@ -25,7 +25,7 @@ export const ProviderVirtualMachinesList: React.FC<{
   });
 
   const [vmData, vmDataLoading] = useInventoryVms({ provider }, providerLoaded, providerLoadError);
-  const obj = { provider, vmData, vmDataLoading: vmDataLoading || vmData?.length === 0 };
+  const obj = { provider, vmData, vmDataLoading: vmDataLoading };
 
   return (
     <ProviderVirtualMachinesListWrapper

diff --git a/packages/forklift-console-plugin/src/modules/Providers/hooks/useProviderInventory.ts b/packages/forklift-console-plugin/src/modules/Providers/hooks/useProviderInventory.ts
@@ -77,6 +77,8 @@ export const useProviderInventory = <T>({
 
   // Fetch data from API
   useEffect(() => {
+    setLoading(true);
+
     const fetchData = async () => {
       if (disabled) {
         return;