Merge pull request #82 from indec-it/fix/currencySchemaType

fix(buildYupSchema): change currency to number
indec-it · Aug 29, 2023 · c9dc3d8 · c9dc3d8
2 parents 35869b3 + 92f3920
commit c9dc3d8
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@indec/form-builder",
-  "version": "2.4.1",
+  "version": "2.4.3",
   "description": "Form builder",
   "main": "index.js",
   "private": false,

diff --git a/src/components/Currency/Currency.js b/src/components/Currency/Currency.js
@@ -7,7 +7,7 @@ import TextField from '../TextField';
 function Currency({form, field, label, ...props}) {
   const handleChange = values => {
     const {floatValue} = values;
-    form.setFieldValue(field.name, floatValue);
+    form.setFieldValue(field.name, typeof floatValue === 'number' ? floatValue : '');
     form.setFieldTouched(field.name, false);
   };
 

diff --git a/src/utils/buildYupSchema.js b/src/utils/buildYupSchema.js
@@ -11,8 +11,8 @@ const getValidatorType = (type, options, metadata) => {
   case questionTypes.TEXT_FIELD:
   case questionTypes.DROPDOWN:
   case questionTypes.RADIO:
-  case questionTypes.CURRENCY:
     return Yup.string().default('');
+  case questionTypes.CURRENCY:
   case questionTypes.NUMERIC_FIELD:
     return Yup.number()
       .transform(value => (Number.isNaN(value) || value === null || value === undefined || value === '' ? 0 : value));