add isLoading prop for holdings

folio-org · Aug 30, 2024 · 0fd3c5f · 0fd3c5f
1 parent 60b4433
commit 0fd3c5f
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/lib/ConsortiumFieldInventory/ConsortiumFieldInventory.js b/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({

diff --git a/lib/FieldHolding/FieldHolding.js b/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 });
 
@@ -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,

diff --git a/lib/FieldInventory/FieldInventory.js b/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,