Skip to content

Commit 3b6e2bf

Browse files
Merge pull request #1 from abdulbasithqb/fixes/validator-allow-emty-string
Fixes/validator allow emty string
2 parents 35d1849 + 7c0ce0b commit 3b6e2bf

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

src/components/textField/validators.ts

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,15 @@ const priceRegEx = /^[0-9]{1,9}([.][0-9]{1,2})?$/;
88

99
const validators = {
1010
required: (value = '') => !_.isEmpty(value),
11-
email: (value = '') => new EmailValidator().isValid(value),
11+
email: (value = '') => {
12+
// Allow empty strings
13+
if (_.isEmpty(value)) {
14+
return true;
15+
}
16+
return new EmailValidator().isValid(value);
17+
},
1218
url: (value = '') => urlRegEx.test(value),
13-
number: (value = '') => integerRegEx.test(value) || decimalNumberRegEx.test(value),
19+
number: (value = '') => _.isEmpty(value) || integerRegEx.test(value) || decimalNumberRegEx.test(value),
1420
price: (value = '') => priceRegEx.test(value)
1521
};
1622

0 commit comments

Comments
 (0)