Merge pull request #5938 from Expensify/luke-min-phone-length-vba

Added minimum phone number length
Expensify · Oct 19, 2021 · 9dc6877 · 9dc6877
2 parents 88766ac + aebe59f
commit 9dc6877
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/CONST.js b/src/CONST.js
@@ -427,6 +427,7 @@ const CONST = {
         DEFAULT: 'default',
     },
     PHONE_MAX_LENGTH: 15,
+    PHONE_MIN_LENGTH: 5,
     REGEX: {
         US_PHONE: /^\+1\d{10}$/,
         DIGITS_AND_PLUS: /^\+?[0-9]*$/,

diff --git a/src/libs/ValidationUtils.js b/src/libs/ValidationUtils.js
@@ -154,7 +154,7 @@ function meetsAgeRequirements(date) {
  * @returns {Boolean}
  */
 function isValidPhoneWithSpecialChars(phoneNumber) {
-    return CONST.REGEX.PHONE_WITH_SPECIAL_CHARS.test(phoneNumber) && phoneNumber.length <= CONST.PHONE_MAX_LENGTH;
+    return CONST.REGEX.PHONE_WITH_SPECIAL_CHARS.test(phoneNumber) && phoneNumber.length <= CONST.PHONE_MAX_LENGTH && phoneNumber.length >= CONST.PHONE_MIN_LENGTH;
 }
 
 /**