[CHECKLIST?][HOLD for payment 2024-05-03] [$250] Complete the final cleanup for distance request refactoring

[tgolen]

This is a part of #29107. You can look at that issue for more context behind the cleanup process.

Currently, we migrated the request flow to the new refactor and there are some deprecated things that need to be cleaned up

• Remove MONEY_REQUEST_MERCHANT route (There are no pages related to this route)
• Remove EditRequestReceiptPage component and related route
• Edit IOU.js and remove _temporaryForRefactor from the name of any methods
• Remove ONYXKEYS.IOU (also need to remove some files that relate to IOU like src/pages/iou/propTypes/index.js)
• Remove EditSplitBillPage (also should remove EditRequestTagPage)
• I suggest we also should move src/pages/iou/steps/MoneyRequestAmountForm.tsx to src/pages/iou/
• We also need to remove MoneyRequestWaypointPage

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01fe66ac3d8b2232c0
• Upwork Job ID: 1783102152724148224
• Last Price Increase: 2024-04-24

Issue Owner

Current Issue Owner: @jliexpensify

[melvin-bot]

Current assignee @DylanDylann is eligible for the External assigner, not assigning anyone new.

[melvin-bot]

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

[tgolen]

@brunovjk Can you please comment on this issue so I can assign it to you? We already spoke about your working on this one.

[brunovjk]

Awesome!

[tgolen]

Hm, I'm not sure why it didn't create an upwork job 🤔 @bfitzexpensify do you know why?

[melvin-bot]

⚠️ Could not update price automatically because there is no linked Upwork Job ID. The BZ team member will need to update the price manually in Upwork.

[brunovjk]

Thanks @tgolen, I already had a proposal ready, I posted it just as a record, no problem? I will start working in PR this afternoon, ETA until 04/30, hopefully sooner.

[melvin-bot]

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

[melvin-bot]

Current assignee @DylanDylann is eligible for the External assigner, not assigning anyone new.

[tgolen]

Sure, thanks! Looks like I got the upwork job created now.

[DylanDylann]

@brunovjk

ETA until 04/30, hopefully sooner.

This is so straightforward. Could you complete PR sooner? Please ping me if there is any problem

[brunovjk]

Sure! You're right, I'll try to get it ready for today.

[jliexpensify]

Invited everyone, Payment Summary:

• C: @brunovjk $250
• C+: @DylanDylann $250

Upworks Job

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.66-5 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• [No QA]: Complete the final cleanup for distance request refactoring #40896

If no regressions arise, payment will be issued on 2024-05-03. 🎊

For reference, here are some details about the assignees on this issue:

• @brunovjk requires payment (Needs manual offer from BZ)
• @DylanDylann requires payment (Needs manual offer from BZ)

[melvin-bot]

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

• [@DylanDylann] The PR that introduced the bug has been identified. Link to the PR:
• [@DylanDylann] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
• [@DylanDylann] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
• [@DylanDylann] Determine if we should create a regression test for this bug.
• [@DylanDylann] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
• [@jliexpensify] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

[melvin-bot]

Payment Summary

Upwork Job

• ROLE: @brunovjk paid $(AMOUNT) via Upwork (LINK)
• ROLE: @DylanDylann paid $(AMOUNT) via Upwork (LINK)

BugZero Checklist (@jliexpensify)

• I have verified the correct assignees and roles are listed above and updated the neccesary manual offers
• I have verified that there are no duplicate or incorrect contracts on Upwork for this job (https://www.upwork.com/ab/applicants/1783102152724148224/hired)
• I have paid out the Upwork contracts or cancelled the ones that are incorrect
• I have verified the payment summary above is correct

[jliexpensify]

@DylanDylann any need for a checklist?

[jliexpensify]

Everyone paid and job removed.

[DylanDylann]

@jliexpensify the checklist is unnecessary. Let's close

[jliexpensify]

Thanks!