Repository navigation
[$250] Migrate index.native.js to function component #16187
Description
Class Component Migration
Filenames
- src/components/PDFView/index.native.js
- type:
React.Component - has state values: true
- has refs: false
- has context: false
- lifecycle methods:
componentDidUpdate
- type:
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/~0157e58f6b76a63a6f
- Upwork Job ID: 1691722444820840448
- Last Price Increase: 2023-08-16
Activity
[-][HOLD] Migrate index.native.js to function component[/-][+][HOLD][$250] Migrate index.native.js to function component[/+]MelvinBot commentedon Apr 13, 2023
marcaaron commentedon Apr 13, 2023
Heads up! The pricing for this issue has been adjusted based on the scope of the work and the fact that no complex bug investigations or proposal are required.
dummy-1111 commentedon Jun 6, 2023
I'd love to work on this.
kadiealexander commentedon Jul 6, 2023
I have this queued to work on, not overdue.
olexyt commentedon Jul 7, 2023
I can work on this issue.
48 remaining items
marcaaron commentedon Nov 22, 2023
Looks like this should be finished before the EOY
melvin-bot commentedon Dec 6, 2023
This issue has not been updated in over 15 days. @kadiealexander, @0xmiroslav eroding to Monthly issue.
P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!
melvin-bot commentedon Jan 24, 2024
Triggered auto assignment to Contributor Plus for review of internal employee PR - @Santhosh-Sellavel (
Internal)mallenexpensify commentedon Jan 24, 2024
@Santhosh-Sellavel reassigning, please take over as C+. If you don't have bandwidth, unassign yourself. Thanks
kadiealexander commentedon Jan 24, 2024
This is sitting with @MonilBhavsar for final review, so it's possible no more c+ action is needed.
melvin-bot commentedon Apr 5, 2024
@kadiealexander, @Santhosh-Sellavel, this Monthly task hasn't been acted upon in 6 weeks; closing.
If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.