App: [HOLD for payment 2023-08-10] [$250] Migrate index.js to function component

Class Component Migration

Filenames

  • src/components/Modal/index.js
    • type: React.Component
    • has state values: false
    • has refs: true
    • has context: false
    • lifecycle methods: componentDidMount,componentWillUnmount

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/~0171f2a01c70848034
  • Upwork Job ID: 1682085982094520320
  • Last Price Increase: 2023-07-20

About this issue

  • Original URL
  • State: closed
  • Created a year ago
  • Comments: 49 (26 by maintainers)

Most upvoted comments

Sorry for the delay 🙏 @Christinadobrzyn We don’t need one.

@Christinadobrzyn tapping you in here since I will be OOO next week. Thanks!

I am also going OOO for next week and taking a similar approach as @michaelhaxhiu here…

No proposals are necessary. If you want to work on an issue make sure that it is off HOLD and has the External + Help Wanted label applied. The first to leave a comment after the issue is taken off HOLD will most likely get assigned to the issue, but Expensify will use our discretion here in case there is someone we’d rather have the migration done by due to experience, overlap with another project, etc.

https://expensify.slack.com/archives/C01GTK53T8Q/p1685727880152119