Set loading when changing provider

Signed-off-by: yaacov <[email protected]>
kubev2v · Jun 25, 2024 · 562327a · 562327a
1 parent 4750252
commit 562327a
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;