fix: 1634 & 1640 A few bug fixes (#1673)

Co-authored-by: Xiao Peng <[email protected]>
bcgov · Nov 18, 2024 · 7367346 · 7367346
1 parent f84d1a9
commit 7367346
Show file tree

Hide file tree

Showing 10 changed files with 34 additions and 29 deletions.
diff --git a/.gitignore b/.gitignore
@@ -210,4 +210,4 @@ schemaspy/output
 # local testing
 *.bat
 
-oracle-api/config
+oracle-api/config
diff --git a/docker-compose.yml b/docker-compose.yml
@@ -42,7 +42,7 @@ services:
     environment:
       ALLOWED_ORIGINS: "http://localhost:*"
       FORESTCLIENTAPI_ADDRESS: https://nr-forest-client-api-prod.api.gov.bc.ca/api
-      FORESTCLIENTAPI_KEY: "${FORESTCLIENTAPI_KEY}"
+      FORESTCLIENTAPI_KEY: YrmI6eO6M6Xeyw2oRYuDaueYPDOgyBuA
       ORACLE_SERVER_URL: https://nr-spar-test-oracle-api.apps.silver.devops.gov.bc.ca
       POSTGRES_HOST: database
       AWS_COGNITO_ISSUER_URI: "https://cognito-idp.ca-central-1.amazonaws.com/ca-central-1_t2HSZBHur"

diff --git a/frontend/src/components/ClientAndCodeInput/definitions.ts b/frontend/src/components/ClientAndCodeInput/definitions.ts
@@ -18,6 +18,7 @@ type ClientAndCodeInputProps = {
   readOnly?: boolean,
   maxInputColSize?: number,
   checkBoxInput?: BooleanInputType
+  shouldSelectDefaultValue?: boolean
 }
 
 export default ClientAndCodeInputProps;
diff --git a/frontend/src/components/ClientAndCodeInput/index.tsx b/frontend/src/components/ClientAndCodeInput/index.tsx
@@ -26,7 +26,7 @@ import './styles.scss';
 const ClientAndCodeInput = ({
   checkboxId, clientInput, locationCodeInput, textConfig, defaultClientNumber,
   defaultLocCode, setClientAndCode, readOnly, showCheckbox, maxInputColSize,
-  checkBoxInput
+  checkBoxInput, shouldSelectDefaultValue = false
 }: ClientAndCodeInputProps) => {
   const getIsDefaultVal = () => (
     checkBoxInput === undefined
@@ -38,9 +38,16 @@ const ClientAndCodeInput = ({
   const clientInputRef = useRef<HTMLInputElement>(null);
   const locCodeInputRef = useRef<HTMLInputElement>(null);
   const [isDefault, setIsDefault] = useState<boolean>(
-    () => getIsDefaultVal()
+    !shouldSelectDefaultValue ? false : () => getIsDefaultVal()
   );
 
+  const [isChecked, setIsChecked] = useState<boolean>(() => {
+    if (!shouldSelectDefaultValue) {
+      return false;
+    }
+    return checkBoxInput ? checkBoxInput.value : isDefault;
+  });
+
   const [showClientValidationStatus, setShowClientValidationStatus] = useState<boolean>(true);
 
   const [showLocCodeValidationStatus, setShowLocCodeValidationStatus] = useState<boolean>(false);
@@ -60,7 +67,9 @@ const ClientAndCodeInput = ({
   useEffect(() => {
     const areValsDefault = getIsDefaultVal();
 
-    setIsDefault(areValsDefault);
+    if (shouldSelectDefaultValue) {
+      setIsDefault(areValsDefault);
+    }
 
     // Do not show validation status if isDefault is true
     if (areValsDefault) {
@@ -223,10 +232,8 @@ const ClientAndCodeInput = ({
         }
         : undefined
     );
-
-    if (!checkBoxInput) {
-      setIsDefault(checked);
-    }
+    setIsDefault(checked);
+    setIsChecked(checked);
   };
 
   const [openAgnTooltip, setOpenAgnTooltip] = useState<boolean>(false);
@@ -343,7 +350,7 @@ const ClientAndCodeInput = ({
                   name={textConfig.useDefaultCheckbox.name}
                   labelText={textConfig.useDefaultCheckbox.labelText}
                   readOnly={readOnly}
-                  checked={checkBoxInput ? checkBoxInput.value : isDefault}
+                  checked={isChecked}
                   onChange={(e: React.ChangeEvent<HTMLInputElement>) => {
                     handleDefaultCheckBox(e.target.checked);
                   }}
@@ -363,7 +370,7 @@ const ClientAndCodeInput = ({
               id={clientInput.id}
               autoCapitalize="on"
               labelText={textConfig.agencyInput.titleText}
-              defaultValue={forestClientQuery.data?.acronym}
+              defaultValue={shouldSelectDefaultValue ? forestClientQuery.data?.acronym : ''}
               helperText={
                 (readOnly || (showCheckbox && isDefault))
                   ? null
@@ -406,7 +413,7 @@ const ClientAndCodeInput = ({
               id={locationCodeInput.id}
               ref={locCodeInputRef}
               name={textConfig.locationCode.name}
-              defaultValue={locationCodeInput.value}
+              defaultValue={shouldSelectDefaultValue ? locationCodeInput.value : ''}
               type="number"
               maxCount={LOCATION_CODE_LIMIT}
               enableCounter

diff --git a/frontend/src/components/LotApplicantAndInfoForm/index.tsx b/frontend/src/components/LotApplicantAndInfoForm/index.tsx
@@ -118,6 +118,7 @@ const LotApplicantAndInfoForm = ({
         }
         readOnly={isEdit}
         maxInputColSize={6}
+        shouldSelectDefaultValue
       />
       <Row className="agency-email-row">
         <Column sm={4} md={8} lg={16} xlg={12}>

diff --git a/frontend/src/components/SeedlotRegistrationSteps/CollectionStep/constants.ts b/frontend/src/components/SeedlotRegistrationSteps/CollectionStep/constants.ts
@@ -49,12 +49,12 @@ export const fieldsConfig = {
   },
   volumePerContainers: {
     name: 'volumePerContainers',
-    labelText: 'Volume per Containers (HI)',
+    labelText: 'Volume per containers (hl)',
     invalidText: 'Invalid entry. Number must be between 0 and 10,000 and up to 3 decimal places.'
   },
   volumeOfCones: {
     name: 'volumeOfCones',
-    labelText: 'Volume of Cones (HI)',
+    labelText: 'Volume of cones (hl)',
     invalidText: 'Number has more than 3 decimals.',
     helperText: 'This value must be the "Volume per container" X "Number of containers".',
     warnText: 'The total volume of cones does not equal, please note that this value must be the "Volume per container" x "Number of containers"'

diff --git a/frontend/src/components/SeedlotRegistrationSteps/ParentTreeStep/utils.ts b/frontend/src/components/SeedlotRegistrationSteps/ParentTreeStep/utils.ts
@@ -920,15 +920,11 @@ export const isMissingSecondaryOrchard = (orchardStepData: OrchardForm): boolean
  * Check if orchards selections are valid.
  */
 export const areOrchardsValid = (orchardStepData: OrchardForm): boolean => {
-  let isValid = true;
   const { orchards } = orchardStepData;
 
   if (!orchards.primaryOrchard.value.code) {
-    isValid = false;
-  }
-  if (isMissingSecondaryOrchard(orchardStepData)) {
-    isValid = false;
+    return false;
   }
 
-  return isValid;
+  return true;
 };
diff --git a/frontend/src/types/SeedlotType.ts b/frontend/src/types/SeedlotType.ts
@@ -207,9 +207,9 @@ export type CollectionFormSubmitType = {
   collectionLocnCode: string,
   collectionStartDate: string,
   collectionEndDate: string,
-  noOfContainers: number,
-  volPerContainer: number,
-  clctnVolume: number,
+  noOfContainers: string,
+  volPerContainer: string,
+  clctnVolume: string,
   seedlotComment: string,
   coneCollectionMethodCodes: Array<number>
 }

diff --git a/frontend/src/views/Seedlot/ContextContainerClassA/constants.tsx b/frontend/src/views/Seedlot/ContextContainerClassA/constants.tsx
@@ -120,9 +120,9 @@ export const emptyCollectionStep: CollectionFormSubmitType = {
   collectionLocnCode: '',
   collectionStartDate: '',
   collectionEndDate: '',
-  noOfContainers: 1,
-  volPerContainer: 1,
-  clctnVolume: 1,
+  noOfContainers: '',
+  volPerContainer: '',
+  clctnVolume: '',
   seedlotComment: '',
   coneCollectionMethodCodes: []
 };

diff --git a/frontend/src/views/Seedlot/ContextContainerClassA/utils.ts b/frontend/src/views/Seedlot/ContextContainerClassA/utils.ts
@@ -852,9 +852,9 @@ export const convertCollection = (collectionData: CollectionForm): CollectionFor
   // Assume the date values are present as validation has occurred before payload is generated
   collectionStartDate: localDateToUtcFormat(collectionData.startDate.value)!,
   collectionEndDate: localDateToUtcFormat(collectionData.endDate.value)!,
-  noOfContainers: +collectionData.numberOfContainers.value,
-  volPerContainer: +collectionData.volumePerContainers.value,
-  clctnVolume: +collectionData.volumeOfCones.value,
+  noOfContainers: `${+collectionData.numberOfContainers.value}`,
+  volPerContainer: `${+collectionData.volumePerContainers.value}`,
+  clctnVolume: `${+collectionData.volumeOfCones.value}`,
   seedlotComment: collectionData.comments.value,
   coneCollectionMethodCodes: collectionData
     .selectedCollectionCodes.value.map((code) => parseInt(code, 10))