App: [HOLD for payment 2023-07-14] [$250] Migrate MoneyRequestParticipantsSplitSelector.js to function component
Class Component Migration
Filenames
Task
- We currently have some class components in our codebase that we would like to refactor to a function component.
- Here’s a link with some general advice on how to refactor a class component to a function component: https://react.dev/reference/react/Component#alternatives
- If you need additional guidance, please ask in #expensify-open-source
- Test for any regressions and verify that there are no breaking changes
Upwork Automation - Do Not Edit
- Upwork Job URL: https://www.upwork.com/jobs/~01a5adde5530f45b97
- Upwork Job ID: 1665751679805845504
- Last Price Increase: 2023-06-05
About this issue
- Original URL
- State: closed
- Created a year ago
- Comments: 46 (29 by maintainers)
Thanks @multijump! I’ve paid you $250 through Upwork. Going to close this GH and the Upwork job posting but please reopen if I’ve missed something!
Hi, @Christinadobrzyn I accepted your offer. Thanks.
Thanks @multijump - sent an offer to you, if you can accept it, I’ll pay this out!
@allroundexperts is paid
Hi, @Christinadobrzyn https://www.upwork.com/freelancers/~0168534bdf28bd4cf8 Here is my upwork profile. Thanks.
@Christinadobrzyn I’ll be getting paid through the app.
okay hiring @allroundexperts as C+ and @multijump as contributor.
Upwork Internal posting - https://www.upwork.com/ab/applicants/1681244482343641088/job-details Upwork External posting - https://www.upwork.com/jobs/~01ced557c4c463eb04
@multijump can you provide your Upwork profile link so I can invite you?
Hi @Christinadobrzyn, this issue is ready to be paid when you’re back from OOO.
Hi @Julesssss @Christinadobrzyn This issue was fully completed?
Yeah, just to clarify we closed the linked issue as expected behavior.
Nop. All good.
im out sick. @allroundexperts on c+ channel said that they’re taking over. Thanks!
https://github.com/Expensify/App/pull/20271#issuecomment-1586352415
@Julesssss I created the PR https://github.com/Expensify/App/pull/20271 Please give me your feedback.
Sorry my mistake. I’m going to assign @multijump for this task.
I tried but this was a tricky refactor for my first attempt, specifically the large
getSectionsfunction was where I started to fail.@multijump I’d be happy to assign this to you while I pick up a simpler one for my first migration.