Illiar/fix/v2 adjustments (#3007)

* fix: adjusted delays & fixed date picker events

* fix: adjusted validator params & fixed revision

* feat: bumped ui & lock fix

apps/kyb-app/src/pages/CollectionFlow/versions/v2/components/organisms/CollectionFlowUI/CollectionFlowUI.tsx

@@ -21,9 +21,9 @@ interface ICollectionFlowUIProps<TValues = CollectionFlowContext> {
 }
 
 const validationParams = {
-  validateOnBlur: true,
-  abortEarly: true,
-  validationDelay: 200,
+  validateOnBlur: false,
+  abortEarly: false,
+  validationDelay: 500,
 };
 
 export const CollectionFlowUI: FunctionComponent<ICollectionFlowUIProps> = ({

apps/kyb-app/src/pages/CollectionFlow/versions/v2/components/organisms/CollectionFlowUI/hooks/usePriorityFields/utils/generate-priority-fields.ts

@@ -24,7 +24,7 @@ export const generatePriorityFields = (
       // Extracting revision reason fro documents isnt common so we handling it explicitly
       if (isDocumentFieldDefinition(element)) {
         const documents = get(context, formatValueDestination(element.valueDestination, stack));
-        const document = documents.find(
+        const document = documents?.find(
           (doc: TDocument) => doc.id === element.params?.template?.id,
         );
 

packages/ui/CHANGELOG.md

@@ -4,6 +4,7 @@
 
 ### Patch Changes
 
+- Param adjustmetns & bugfixes
 - Format ongoing summary in the UI
 
 ## 0.5.69

packages/ui/src/components/molecules/inputs/DatePickerInput/DatePickerInput.tsx

@@ -168,6 +168,8 @@ export const DatePickerInput = ({
               ),
               inputProps: {
                 'data-test-id': testId,
+                onBlur: onBlur,
+                onFocus: onFocus,
               },
               InputProps: {
                 className: 'focus:outline-none',

packages/ui/src/components/organisms/Form/DynamicForm/fields/DocumentField/hooks/useDocumentUpload/helpers/create-or-update-fileid-or-file-in-documents/create-or-update-fileid-or-file-in-documents.ts

@@ -28,6 +28,7 @@ export const createOrUpdateFileIdOrFileInDocuments = (
     return documents;
   } else {
     const existingDocumentIndex = documents.findIndex(document => document.id === template?.id);
+    documents[existingDocumentIndex] = structuredClone(template);
     const pathToFileId = composePathToFileId(existingDocumentIndex, pageProperty, pageIndex);
 
     set(documents, pathToFileId, fileIdOrFile);

packages/ui/src/components/organisms/Form/DynamicForm/hooks/external/useField/useField.ts

@@ -26,7 +26,7 @@ export const useField = <TValue>(element: IFormElement<any, any>, stack?: TDeept
   const disabledRulesResult = useRuleEngine(valuesAndMetadata, {
     rules: useRules(element.disable, stack),
     runOnInitialize: true,
-    executionDelay: 500,
+    executionDelay: 100,
   });
 
   const isDisabled = useMemo(() => {
@@ -38,13 +38,12 @@ export const useField = <TValue>(element: IFormElement<any, any>, stack?: TDeept
   const onChange = useCallback(
     <TValue>(value: TValue, ignoreEvent = false) => {
       setValue(fieldId, valueDestination, value);
-      setTouched(fieldId, true);
 
       if (!ignoreEvent) {
         sendEventAsync('onChange');
       }
     },
-    [fieldId, valueDestination, setValue, setTouched, sendEventAsync],
+    [fieldId, valueDestination, setValue, sendEventAsync],
   );
 
   const onBlur = useCallback(() => {

packages/ui/src/components/organisms/Form/DynamicForm/hooks/external/useField/useField.unit.test.ts

@@ -155,7 +155,6 @@ describe('useField', () => {
       result.current.onChange('new-value');
 
       expect(mockSetValue).toHaveBeenCalledWith('test-field-1-2', 'test.path[1][2]', 'new-value');
-      expect(mockSetTouched).toHaveBeenCalledWith('test-field-1-2', true);
       expect(mockSendEventAsync).toHaveBeenCalledWith('onChange');
     });
 
@@ -274,7 +273,7 @@ describe('useField', () => {
         {
           rules: mockElement.disable,
           runOnInitialize: true,
-          executionDelay: 500,
+          executionDelay: 100,
         },
       );
     });
@@ -296,7 +295,7 @@ describe('useField', () => {
         {
           rules: mockElement.disable,
           runOnInitialize: true,
-          executionDelay: 500,
+          executionDelay: 100,
         },
       );
     });