feat(locations): combine metrics and bucket-list loading/error states

Part of the implementation plan from scality/agent-task#343.

Author: Cerebro

src/react/next-architecture/domain/business/locations.ts

@@ -160,14 +160,12 @@ export const useListLocationsForCurrentAccount = ({
     enabled: !!accountCannonicalId,
   });
 
-  const {
-    bucketList,
-    status: bucketListStatus,
-    isFetching: bucketListIsFetching,
-    error: bucketListError,
-  } = useBucketList();
+  const { bucketList, status: bucketListStatus } = useBucketList();
 
   const bucketLocationIds = useMemo(() => {
+    if (bucketListStatus === 'error') {
+      return [];
+    }
     return Array.from(
       new Set(
         bucketList
@@ -176,7 +174,7 @@ export const useListLocationsForCurrentAccount = ({
           .filter((location): location is string => !!location),
       ),
     );
-  }, [bucketList, accountCannonicalId]);
+  }, [bucketList, accountCannonicalId, bucketListStatus]);
 
   if (account === undefined) {
     return {
@@ -215,6 +213,14 @@ export const useListLocationsForCurrentAccount = ({
     };
   }
 
+  if (bucketListStatus === 'idle' || bucketListStatus === 'loading') {
+    return {
+      locations: {
+        status: 'loading',
+      },
+    };
+  }
+
   const accountLocationsKey = Array.from(new Set([...Object.keys(accountLocationData), ...bucketLocationIds]));
 
   if (allLocations.locations.status !== 'success') {