Disable autocorrect on international bank account form inputs#89533
Disable autocorrect on international bank account form inputs#89533
Conversation
… text inputs
iOS autocorrect silently modifies structured financial data (account numbers,
IFSC codes, SWIFT codes) in the international deposit account forms. This adds
autoCorrect={false} and spellCheck={false} to TextInput fields across all five
affected bank form pages, applied conditionally so non-text components
(ValuePicker, AddressSearch, PushRowWithModal) are unaffected.
Co-authored-by: truph01 <truph01@users.noreply.github.com>
…l props Co-authored-by: truph01 <truph01@users.noreply.github.com>
|
Fixed the failing ESLint check: replaced prop spreading ( Note: The |
Codecov Report✅ Changes either increased or maintained existing code coverage, great job!
|
|
The failing ESLint check is unrelated to this PR's changes. Failure: Evidence: This PR only modifies bank account form pages ( The other failing check ( |
Explanation of Change
iOS autocorrect is enabled by default on native
TextInputfields. For structured financial data like account numbers, IFSC codes, and SWIFT/BIC codes, autocorrect silently modifies values (e.g., "Whats" → "What's") after step-level validation has passed. The corrupted values then reach the Corpay API, which rejects them with no clear indication of what went wrong.This PR adds
autoCorrect={false}andspellCheck={false}toTextInputfields in all five international bank account form pages. The props are applied conditionally so that non-text-input components (ValuePicker,AddressSearch,PushRowWithModal) are unaffected.Affected pages:
BankAccountDetails.tsx(personal wallet flow)BankInformation.tsx(personal wallet flow)AccountHolderInformation.tsx(personal wallet flow)NonUSD/BankInfo/subSteps/BankAccountDetails.tsx(workspace flow)NonUSD/BankInfo/subSteps/AccountHolderDetails.tsx(workspace flow)Fixed Issues
$ #84811
PROPOSAL: #84811 (comment)
Tests
// TODO: The human co-author must fill out the tests you ran before marking this PR as "ready for review"
// Please describe what tests you performed that validates your changed worked.
Offline tests
N/A — this change only disables iOS autocorrect/spellcheck on input fields and does not affect network behavior.
QA Steps
// TODO: The human co-author must fill out the QA tests you ran before marking this PR as "ready for review".
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectiontoggleReportand notonIconClick)src/languages/*files and using the translation methodSTYLE.md) were followedAvatar, I verified the components usingAvatarare working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))npm run compress-svg)Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.ScrollViewcomponent to make it scrollable when more elements are added to the page.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari