[HOLD for payment 2023-07-17] [$250] Migrate ReportActionsView.js to function component

[marcaaron]

Class Component Migration

Filenames

• src/pages/home/report/ReportActionsView.js
  • type: React.Component
  • has state values: true
  • has refs: true
  • has context: false
  • lifecycle methods: componentDidMount,componentWillUnmount,componentDidUpdate,shouldComponentUpdate

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/~01a5211f387b070380
• Upwork Job ID: 1673399672865730560
• Last Price Increase: 2023-06-26

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

[muxriddinmuqimov77]

I'd like to work on this issue

[melvin-bot]

Triggered auto assignment to @muttmuure (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details.

[melvin-bot]

Triggered auto assignment to Design team member for new feature review - @shawnborton (NewFeature)

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~01a5211f387b070380

[melvin-bot]

Current assignee @muttmuure is eligible for the External assigner, not assigning anyone new.

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @narefyev91 (External)

[Vishrut19]

Dibs?

[rayane-d]

can I work on this?

[olexyt]

I love to work on this issue

[asadath1395]

I would love to work on this. Can i get this assigned?

[muttmuure]

We prefer if people focus on 1 issue at a time, so I'm going to assign @rayane-djouah.

[melvin-bot]

📣 @rayane-djouah 🎉 An offer has been automatically sent to your Upwork account 🎉

Contributor - [$250] Migrate ReportActionsView.js to function component Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

[rayane-d]

@muttmuure, can we address this issue regarding the payment amount? I have discovered that this particular component refactor requires a significant amount of work compared to other React class component migration issues.

[rayane-d]

@muttmuure

[rayane-d]

PR ready for review.

[rayane-d]

@muttmuure @narefyev91 could you review the PR please?

[bernhardoj]

@rayane-djouah please apply the suggestion here #16267 (comment)

[rayane-d]

@bernhardoj I applied your suggestion, thanks!

[rayane-d]

@narefyev91 could you review the PR please?

[muttmuure]

@rayane-djouah feel free to DM me in Slack with more details and I'll chat with the team about this

[melvin-bot]

Triggered auto assignment to @neil-marcellini, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

[rayane-d]

@neil-marcellini could you review the PR please?

[rayane-d]

PR merged

[melvin-bot]

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

[melvin-bot]

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

[melvin-bot]

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

• Migrate ReportActionsView.js to function component #22200

If no regressions arise, payment will be issued on 2023-07-17. 🎊

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

[muttmuure]

All done.