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)

Most upvoted comments

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!

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 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.

⚠️ Looks like this issue was linked to a Deploy Blocker here If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results. If a regression has occurred and you are the assigned CM follow the instructions here. If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

Hello @allroundexperts @Christinadobrzyn So everything is fine now? Or there is something which I should change?

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 getSections function 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.