IOS - VBA - Unable to add alphabets in account number

[kbecciv]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Action Performed:

1. Open app
2. Navigate to settings > Workspace > Add bank account
3. Proceed to the manual flow
4. Add alphabets in account number

Expected Result:

User should be able to add alphanumeric characters in account number field

Actual Result:

Unabe able to add alphanumeric characters in account number field. Only numeric keyboard opens up

Workaround:

Unknown

Platform:

Where is this issue occurring?

• Web
• iOS
• Android
• Desktop App
• Mobile Web

Version Number: 1.1.29-1

Reproducible in staging?: Yes

Reproducible in production?: Yes

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:

Issue reported by: Applause

Slack conversation:

View all open jobs on GitHub

Issue found when executing PR #7101

[MelvinBot]

Triggered auto assignment to @bondydaa (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[rushatgabhane]

Not an issue?
Recently changed account numbers to numeric only in #7056

[bondydaa]

I think the title of this is a little confusing but the gist is that you are able to type non-numeric characters into the input but then it just gives you the error that it's an invalid account, I think what this is proposing is that we just don't allow a user to type any non-numeric characters and only accept numbers in the input.

I think that could a decent UX improvement but I don't think it's a huge priority either, I think I'd lean more "do nothing" for now and then if we get user's reporting/complaining about it then we can re-evaluate.

[rushatgabhane]

Ohh, yeah I agree.

[sig5]

Proposal
We can add a validation method that disallow these character via regex.
Changes:

1. InBankAcountStep.js onChangeText can be assigned to the function updateRoutingNumber, which enforces the checks and clearErrorAndSetValuesteps.

[bondydaa]

I'm going to close this out for now, I don't think it's valuable to add since the error message handles it and a user would have to manually toggle to the non-numeric keyboard to enter alphabetic characters.