App: [$1000] Web - Show error when numbers entered in debit card holder name field

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. Go to payments > debit card
  2. Enter a numeric string in name’s place eg: 111111
  3. Fill the remaining details and hit enter

Expected Result:

There should be an error on name field

Actual Result:

No error on name field, 111111 passes the form validation

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android / native
  • Android / Chrome
  • iOS / native
  • iOS / Safari
  • MacOS / Chrome / Safari
  • MacOS / Desktop

Version Number: 1.3.32-5 Reproducible in staging?: y Reproducible in production?: y If this was caught during regression testing, add the test name, ID and link from TestRail: Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Notes/Photos/Videos: Any additional supporting documentation

https://github.com/Expensify/App/assets/93399543/dbc836aa-a606-4c75-9fa0-ca5c5961d784

Expensify/Expensify Issue URL: Issue reported by: https://expensify.slack.com/archives/C049HHMV9SM/p1688813568570969 Slack conversation: @chiragxarora

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~016cce6a5c4931cb3b
  • Upwork Job ID: 1683521297890430976
  • 2023-07-24
  • Automatic offers:
    • | | 0
    • chiragxarora | Contributor | 25763253

About this issue

  • Original URL
  • State: closed
  • Created a year ago
  • Comments: 43 (26 by maintainers)

Most upvoted comments

I agree, time bonus is applicable because the merge delay was on our end, the C+ approval was within 3 business days I’ve updated my comment to show correct numbers

@chiragxarora offer sent for the bonus @rushatgabhane please request $1500 for this issue

This qualifies for time bonus too @sonialiap

Based on my calculations, the pull request did not get merged within 3 working days of assignment. Please, check out my computations here:

  • when @chiragxarora got assigned: 2023-07-26 12:46:45 Z
  • when the PR got merged: 2023-08-03 14:15:29 UTC
  • days elapsed: 6

On to the next one 🚀

Based on the documentation from our vendor, we should be good to go here 👍

Confirmed in slack that the expected behavior is to error when numbers are entered into name field. Triaging to external