1830: Resolves MR comments

Author: Ruben van Leeuwen

frontend/packages/pydantic-forms/src/components/componentMatcher.tsx

@@ -66,6 +66,11 @@ export const componentsMatcher = (
     return components;
 };
 
+const defaultComponent: ElementMatch = {
+    Element: TextField,
+    isControlledElement: true,
+};
+
 export const fieldToComponentMatcher = (
     pydanticFormField: PydanticFormField,
     customComponentMatcher: PydanticFormsContextConfig['componentMatcher'],
@@ -75,12 +80,8 @@ export const fieldToComponentMatcher = (
 
     const ElementMatch: ElementMatch = matchedComponent
         ? matchedComponent.ElementMatch
-        : {
-              Element: TextField,
-              isControlledElement: true,
-          };
+        : defaultComponent; // Defaults to textField when there are no matches
 
-    // Defaults to textField when there are no matches
     return {
         Element: ElementMatch,
         pydanticFormField: pydanticFormField,

frontend/packages/pydantic-forms/src/core/hooks/usePydanticFormParser.tsx

@@ -85,7 +85,7 @@ const getPydanticFormField = (
         attributes: getFieldAttributes(propertySchema),
         schema: propertySchema,
         validations: getFieldValidation(propertySchema),
-        columns: 6, // TODO: Is this still relevant?
+        columns: 6, // TODO: Is this still relevant? https://github.com/workfloworchestrator/orchestrator-ui-library/issues/1891
         properties: parseProperties(
             propertySchema.properties || {},
             propertySchema.required || [],