IOU - Unable to use comma on the amount field

[isagoico]

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. Click on request money
2. In the amount field enter 1,000.00 (or copy and paste the amount)

Expected Result:

User should be able to use a comma in the amount field, or copy and paste the amount with a comma.

Actual Result:

User is unable to paste the amount with a comma or use the comma in the amount field.

Workaround:

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

Platform:

Where is this issue occurring?

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

Version Number: 1.0.88-0

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

Notes/Photos/Videos: Any additional supporting documentation

Grabando.210.mp4

Expensify/Expensify Issue URL:

View all open jobs on GitHub

view this job on upwork

---

From @johnmlee101 https://expensify.slack.com/archives/C01GTK53T8Q/p1630006810047400

When creating an IOU, you cannot paste numbers into the amount field if the amount has a comma.
AKA pasting 1,337.00 doesn't work.

[MelvinBot]

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

[mananjadhav]

Proposal

The regex in the validateAmount function of IOUAmountPage has to be updated.

   validateAmount(amount) {
        const decimalNumberRegex = new RegExp(/^\d+(,\d+)*(\.\d{0,3})?$/, 'i');   //OLD Regex: /^\d+(\.\d{0,3})?$/
        return amount === '' || decimalNumberRegex.test(amount);
    }

While the above regex matches the numbers with comma, we should even remove the commas before sending the amount back.

    if (this.validateAmount(amount)) {
       this.setState({amount: Number((amount + '').replace(/\,/g, ''))}
   }

[MelvinBot]

Triggered auto assignment to @jboniface (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.