App: [$250] Remove MoneyRequestParticipantsPage.js and copy any changes since Nov 27 into IOURequestStepParticipants.js
This is a part of https://github.com/Expensify/App/issues/29107. You can look at that issue for more context behind the cleanup process.
Problem
The app has two redundant components:
Old Component: MoneyRequestParticipantsPage
New Component IOURequestStepParticipants
Solution
Following the examples (example 1, example 2), the Old Component needs to be completely removed from the codebase
- Look at the history of the Old Component
- If there are any changes since Nov 27, 2023 which have not been added to the New Component, copy those changes
- Replace all uses of the Old Component with the New Component
- Remove all traces of Old Component
- Be sure to update all routes and navigation to use the new
:actionparam (instead of being hard-coded with"create") - Update any logic like
isEditingto use the new action param from the route
Upwork Automation - Do Not Edit
- Upwork Job URL: https://www.upwork.com/jobs/~015fc3b79f31f2289a
- Upwork Job ID: 1747632960650227712
- Last Price Increase: 2024-01-24
- Automatic offers:
- abdulrahuman5196 | Reviewer | 28131654
- Krishna2323 | Contributor | 28131655
About this issue
- Original URL
- State: closed
- Created 5 months ago
- Comments: 43 (30 by maintainers)
This is a straightforward task with clear requirements in OP. So approving the first proposal with some information regarding the changes. @Krishna2323 's proposal here https://github.com/Expensify/App/issues/34616#issuecomment-1894614055 looks good and works well.
š š š C+ Reviewed
Yes. Will review proposals today.
@isabelastisser, PR was deployed to production on 24th April, I think this is ready for payments process.
Asked in slack too https://expensify.slack.com/archives/C02NK2DQWUX/p1711752832574839
https://github.com/Expensify/App/pull/35455 is merged, will be raising PR for this very soon.
https://github.com/Expensify/App/pull/35455 is still in progress. Iām the author of that PR also, and we are very close to merging it.
@Krishna2323 I have seen you post proposals on other issues, can you please give ETA for this PR or I will reassign to someone else tomorrow. Thank you