IOU - Uploading a receipt with >24MB freezes the app for a while

[m-natarajan]

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

---

Found when validating #45448
Version Number: 9.0.17-0
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers): nathanmulugetatesting+881@gmail.com
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:

Action Performed:

1. Navigate to staging.new.expensify.com
2. Submit an expense to a user or a workspace
3. Open the expense detail and click on the add receipt button
4. Upload a picture with size > 24MB

Expected Result:

The app quickly closes the RHP and shows navigates to the expense details page (which is the behavior seen for smaller file sizes), or the app should show a loading spinner

Actual Result:

The app freezes for a while when reducing the file size and clicking on anything is not functional

Workaround:

unknown

Platforms:

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

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

Screenshots/Videos

Bug6563182_1722936352565.bandicam_2024-08-06_12-21-16-010.mp4

View all open jobs on GitHub

[melvin-bot]

Triggered auto assignment to @dangrous (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

[melvin-bot]

Triggered auto assignment to @twisterdotcom (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.

[github-actions]

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

1. Identify the pull request that introduced this issue and revert it.
2. Find someone who can quickly fix the issue.
3. Fix the issue yourself.

[twisterdotcom]

I agree this is an issue, but does it need to block the deploy or be hourly? @dangrous what do you think? This isn't a common flow, so I think we could make it just a daily bug.

I know @kidroca recently worked on uploading attachments, but not receipts. Not sure what else could have caused this if it was working well before.

[dangrous]

I thiiiink I agree that this is not a blocker - I believe before the latest deploy you just couldn't upload images greater than 24mb, so this is better than that.

However, we should probably have @luacmartins @nkdengineer @shubham1206agra take a look since it's the new functionality they introduced.

I'm going to demote this, but keep as daily.

[luacmartins]

@nkdengineer @shubham1206agra could you please take a look?

[nkdengineer]

@luacmartins The reason is the resize function takes a long time with the large image. Then I don't think it's a bug.

If we want to improve this case, we can display a loading while the image is uploading. What do you think?

cc @shubham1206agra

[luacmartins]

Agreed! Let's add a loading indicator while things are processing.

[nkdengineer]

@shubham1206agra The PR is here.

cc @luacmartins Please help look at the screenshot and let me know your thoughts.

[dangrous]

I think this went to prod 8/12 but the automation didn't fire. If everything looks good, we can work to close out on 8/19.

Will we need payment for this? or is it regression/followup from #44084

[shubham1206agra]

@dangrous No payment required here

[nkdengineer]

Yes, no payment is required here. It's an improvement from #44084

[dangrous]

Awesome! closing out then.