Skip to content

Commit

Permalink
Merge pull request #40095 from GandalfGwaihir/issue39591
Browse files Browse the repository at this point in the history
[Fix]: `BE` length and `FE` limit for Legal Name are not consistent leading to unsaved input without error after submitting
  • Loading branch information
chiragsalian authored Apr 15, 2024
2 parents a2d1af9 + cac94da commit 95fd760
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/pages/settings/Profile/PersonalDetails/LegalNamePage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -48,10 +48,10 @@ function LegalNamePage({privatePersonalDetails, isLoadingApp = true}: LegalNameP
ErrorUtils.addErrorMessage(errors, 'legalFirstName', 'privatePersonalDetails.error.hasInvalidCharacter');
} else if (!values.legalFirstName) {
errors.legalFirstName = 'common.error.fieldRequired';
} else if (values.legalFirstName.length > CONST.TITLE_CHARACTER_LIMIT) {
} else if (values.legalFirstName.length > CONST.LEGAL_NAME.MAX_LENGTH) {
ErrorUtils.addErrorMessage(errors, 'legalFirstName', [
'common.error.characterLimitExceedCounter',
{length: values.legalFirstName.length, limit: CONST.TITLE_CHARACTER_LIMIT},
{length: values.legalFirstName.length, limit: CONST.LEGAL_NAME.MAX_LENGTH},
]);
}
if (ValidationUtils.doesContainReservedWord(values.legalFirstName, CONST.DISPLAY_NAME.RESERVED_NAMES)) {
Expand All @@ -64,8 +64,8 @@ function LegalNamePage({privatePersonalDetails, isLoadingApp = true}: LegalNameP
ErrorUtils.addErrorMessage(errors, 'legalLastName', 'privatePersonalDetails.error.hasInvalidCharacter');
} else if (!values.legalLastName) {
errors.legalLastName = 'common.error.fieldRequired';
} else if (values.legalLastName.length > CONST.TITLE_CHARACTER_LIMIT) {
ErrorUtils.addErrorMessage(errors, 'legalLastName', ['common.error.characterLimitExceedCounter', {length: values.legalLastName.length, limit: CONST.TITLE_CHARACTER_LIMIT}]);
} else if (values.legalLastName.length > CONST.LEGAL_NAME.MAX_LENGTH) {
ErrorUtils.addErrorMessage(errors, 'legalLastName', ['common.error.characterLimitExceedCounter', {length: values.legalLastName.length, limit: CONST.LEGAL_NAME.MAX_LENGTH}]);
}
if (ValidationUtils.doesContainReservedWord(values.legalLastName, CONST.DISPLAY_NAME.RESERVED_NAMES)) {
ErrorUtils.addErrorMessage(errors, 'legalLastName', 'personalDetails.error.containsReservedWord');
Expand Down

0 comments on commit 95fd760

Please sign in to comment.