[$250] Category CSV - Max Expense Amount in downloaded CSV displays amount without decimal point

[IuliiaHerets]

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

---

Version Number: 9.0.35-0
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers): applausetester+kh010901@applause.expensifail.com
Issue reported by: Applause Internal Team

Action Performed:

Precondition:

• Rules are enabled.

1. Go to staging.new.expensify.com
2. Go to workspace settings > Categories.
3. Click on any category.
4. Click Flag amounts over.
5. Enter amount with decimals (12.34).
6. Save it.
7. Dismiss RHP.
8. Click 3-dot menu > Download CSV.
9. Open the file.

Expected Result:

Max Expense Amount field in the downloaded CSV should display the amount correctly with decimal point.

Actual Result:

Max Expense Amount field in the downloaded CSV displays the amount without decimal point.
When the same category list is downloaded on Old Dot, it does show the amount with decimal point correctly.

Workaround:

Unknown

Platforms:

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

Bug6604536_1726419076572.20240916_004054.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021836046116299332518
• Upwork Job ID: 1836046116299332518
• Last Price Increase: 2024-09-17

Issue Owner

Current Issue Owner: @mollfpr

[melvin-bot]

Triggered auto assignment to @joekaufmanexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

[IuliiaHerets]

@joekaufmanexpensify FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

[joekaufmanexpensify]

Reproduced

[joekaufmanexpensify]

cc @marcaaron for this one too, as I think this is also related to the expense rules doc

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~021836046116299332518

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @mollfpr (External)

[marcaaron]

I'm not familiar with that CSV export feature. We should maybe tag whoever worked on that one?

I'll confirm the backend is working correctly for this in the meantime.

[marcaaron]

So, I think it's indirectly related. There's nothing wrong with the data. We just need to be converting the "cents" values back into "dollars" before exporting to the CSV. Should be an easy fix for whoever knows how this works already.

[marcaaron]

Seems like this issue is where this feature was implemented @rlinoz maybe can help take a look since I think the backend is where this change would need to happen if I'm understanding the problem correctly.

[joekaufmanexpensify]

Got it, sounds good. Thanks Marc!

[rlinoz]

Marc is right, I will take a look!

[joekaufmanexpensify]

Sweet. TY!

[joekaufmanexpensify]

PR(s) in review

[joekaufmanexpensify]

Web PR merged. Second auth PR held on web PR's being deployed

[rlinoz]

This is in prod now