[HOLD for payment 2023-04-24] [$250] Migrate ReportActionsList.js to function component

[marcaaron]

Class Component Migration

Filenames

• src/pages/home/report/ReportActionsList.js
  • type: React.Component
  • has state values: true
  • has refs: true
  • has context: false
  • lifecycle methods: componentDidMount

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

[marcaaron]

@marcaaron taking this issue off HOLD as per this Slack post

IMPORTANT: If anything unexpected pops up during the refactor/review:

• Take it to #expensify-open-source to get clarification on.
• Prefix the conversation with [HOOK REFACTOR] so that we can find it easily.
• Link the result of the conversation in this section of the Design Doc titled “Best Practices Conversations”
• Add it to the STYLE.md guide in this open PR so we can reference it and apply any teachings in the next phase

[s77rt]

@marcaaron Please assign me here and a BZ member for payment processing.

[MelvinBot]

⚠️ Could not update price automatically because there is no linked Upwork Job ID. The BZ team member will need to update the price manually in Upwork.

[marcaaron]

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.

[MelvinBot]

Reviewing label has been removed, please complete the "BugZero Checklist".

[MelvinBot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.3.0-2 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• Remove class component from ReportActionsList #16613

If no regressions arise, payment will be issued on 2023-04-24. 🎊

After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

• External issue reporter
• Contributor that fixed the issue
• Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

• Merged PR within 3 business days of assignment - 50% bonus
• Merged PR more than 9 business days after assignment - 50% penalty

[MelvinBot]

Triggered auto assignment to @greg-schroeder (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

[marcaaron]

@greg-schroeder Added the bug label to get a BZ team member assigned. This isn't a "bug" per se, but part of an internal cleanup. We just need payment for @s77rt 🙇

[marcaaron]

@greg-schroeder bump 🙏

[greg-schroeder]

Sorry about that, taking care of payment

[greg-schroeder]

Sent offer @s77rt

[s77rt]

@greg-schroeder Accepted

[greg-schroeder]

Done!