App: [HOLD #33434][$250] Migrate index.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/~01c18f5687e5c548bf
- Upwork Job ID: 1759601059714211840
- Last Price Increase: 2024-02-19
- Automatic offers:
- s-alves10 | Contributor | 0
- ishpaul777 | Contributor | 0
About this issue
- Original URL
- State: closed
- Created a year ago
- Comments: 52 (39 by maintainers)
@mkhutornyi Thanks! Seems like that https://github.com/Expensify/App/pull/33434 was merged yesterday 😄
I can take as C+ while Santosh is unavailable
I added @Santhosh-Sellavel as C+ since they reviewed https://github.com/Expensify/App/issues/16187
@puneetlath you understood it correctly! 😄
@pac-guerreiro please go ahead. @ishpaul777 can do C+ review on your PR. Btw, are you already aware that this migration is on hold for https://github.com/Expensify/App/pull/33434?
Looks like I am not needed anymore so unassigning, and also I haven’t started yet.
We need to put this on hold for #33434 Reason:
Discussion: https://expensify.slack.com/archives/C02NK2DQWUX/p1710425733927159?thread_ts=1710365206.608239&cid=C02NK2DQWUX
@kadiealexander lmk if you need any help with the VMs!