[HOLD for payment 2023-06-29] [HOLD] [$250] Migrate WorkspaceMembersPage.js to function component

[marcaaron]

Class Component Migration

Filenames

• src/pages/workspace/WorkspaceMembersPage.js
  • type: React.Component
  • has state values: true
  • has refs: false
  • has context: false
  • lifecycle methods: componentDidMount,componentDidUpdate

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

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

[muxriddinmuqimov77]

I can work on this issue when it goes to external

[abdulrahuman5196]

@Li357 Kindly assign me and BZ for C+ review processing.

[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.30-5 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• Refactor WorkspaceMembersPage to functional component #20687

If no regressions arise, payment will be issued on 2023-06-29. 🎊

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

[abdulrahuman5196]

@Li357 Gentle ping on this #16306 (comment)

[abdulrahuman5196]

@Li357 Could you kindly assign the label to assign a BZ on this issue to process payment?

[melvin-bot]

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

[melvin-bot]

Bug0 Triage Checklist (Main S/O)

• This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
• This bug is not a duplicate report (check E/App issues and #expensify-bugs)
  • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
• This bug is reproducible using the reproduction steps in the OP. S/O
  • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
  • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
• This issue is filled out as thoroughly and clearly as possible
  • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
• I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

[abdulrahuman5196]

No BZ Checklist. We are only doing migration to function component

[kevinksullivan]

@abdulrahuman5196 can you apply to this job and let me know wen you've done so?

https://www.upwork.com/jobs/~018a5ff86cf2e66e2c

[Li357]

Not overdue, waiting for Upwork application

[abdulrahuman5196]

@kevinksullivan thank you. Submitted a proposal on the job.

[kevinksullivan]

Thanks @abdulrahuman5196 . One final step on your part is accepting the offer. Let me know when you've done so an I'll wrap this up.

[abdulrahuman5196]

@kevinksullivan thank you. accepted the offer

[kevinksullivan]

Payment sent and contract ended