UISACQCOMP-215: ECS - Add isLoading prop for holdings

CHANGELOG.md

@@ -22,6 +22,7 @@
 * Support using custom list of tenants when open the locations modal. Refs UISACQCOMP-210.
 * ECS - Display all consortium tenants in the affiliation selection of the location lookup. Refs UISACQCOMP-202.
 * Add isMultiSelect prop to DonorsLookup component. Refs UISACQCOMP-212.
+* ECS - Add `isLoading` prop for `ConsortiumFieldInventory` component. Refs UISACQCOMP-215.
 
 ## [5.1.1](https://github.com/folio-org/stripes-acq-components/tree/v5.1.1) (2024-04-22)
 [Full Changelog](https://github.com/folio-org/stripes-acq-components/compare/v5.1.0...v5.1.1)

lib/ConsortiumFieldInventory/ConsortiumFieldInventory.js

@@ -119,6 +119,7 @@ ConsortiumFieldInventory.propTypes = {
   filterLocations: PropTypes.func,
   holdingName: PropTypes.string.isRequired,
   isNonInteractive: PropTypes.bool,
+  isLoading: PropTypes.bool,
   labelless: PropTypes.bool,
   locationName: PropTypes.string.isRequired,
   locations: PropTypes.arrayOf(PropTypes.shape({

lib/FieldHolding/FieldHolding.js

@@ -35,12 +35,13 @@ export const FieldHolding = ({
   name,
   onChange,
   isDisabled = false,
+  isLoading = false,
   isNonInteractive = false,
   required = false,
   tenantId,
 }) => {
   const {
-    isLoading,
+    isLoading: isHoldingsLoading,
     holdings,
   } = useInstanceHoldings(instanceId, { tenantId });
 
@@ -64,7 +65,7 @@ export const FieldHolding = ({
 
       setSelectedLocation(location);
     }
-  }, []);
+  }, [initialHoldingId, locationFieldName, locationsMap, values]);
 
   useEffect(() => {
     const isInitialAffiliation = tenantId === initialTenantId;
@@ -120,7 +121,7 @@ export const FieldHolding = ({
 
   const isLookupTriggerVisible = !(isDisabled || isNonInteractive);
 
-  if (isLoading) return <Loading />;
+  if (isLoading || isHoldingsLoading) return <Loading />;
 
   return (
     selectedLocation
@@ -172,6 +173,7 @@ FieldHolding.propTypes = {
   filterLocations: PropTypes.func,
   instanceId: PropTypes.string.isRequired,
   isDisabled: PropTypes.bool,
+  isLoading: PropTypes.bool,
   isNonInteractive: PropTypes.bool,
   labelId: PropTypes.string,
   locationFieldName: PropTypes.string.isRequired,

lib/FieldInventory/FieldInventory.js

@@ -9,6 +9,7 @@ export const FieldInventory = ({
   filterHoldings,
   filterLocations,
   instanceId,
+  isLoading = false,
   isNonInteractive,
   locations,
   locationIds,
@@ -35,6 +36,7 @@ export const FieldInventory = ({
         locationLabelId={locationLabelId}
         name={holdingName}
         isDisabled={disabled}
+        isLoading={isLoading}
         isNonInteractive={isNonInteractive}
         onChange={onChange}
         required={required}
@@ -77,6 +79,7 @@ FieldInventory.propTypes = {
 
   filterHoldings: PropTypes.func,
   filterLocations: PropTypes.func,
+  isLoading: PropTypes.bool,
   isNonInteractive: PropTypes.bool,
   onChange: PropTypes.func.isRequired,
   required: PropTypes.bool,