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

[marcaaron]

Class Component Migration

Filenames

• src/components/Checkbox.js
  • type: React.Component
  • has state values: true
  • has refs: true
  • has context: false
  • lifecycle methods: none

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]

@kuluruvineeth Please stop spamming the issues in this repo. You are free to comment on things that have the Help Wanted label on them.

[kuluruvineeth]

@marcaaron my apologies, i was not aware that this issues were internal i learnt the same just a while ago. I shall make sure it won't happen again thanks.

[marcaaron]

Yeah, there's not really any good reason to spam a bunch of issues with similar comments in any case. If you are not sure about something ask in Slack and we are happy to provide assistance there, thanks!

[marcaaron]

Heads up, I'm taking this issue off HOLD and making it a Daily. If you are unable to work on it please remove your assignment and add the External + Help Wanted labels as per this update.

[NikkiWines]

Nice, I'll push up a PR for this first thing monday

[kushu7]

I can work on this if it goes to external.

[NikkiWines]

PR is up, working on testing

[olexyt]

I love to work on this issue

[melvin-bot]

@NikkiWines Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[NikkiWines]

PR has been merged

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

• Migrate Checkbox.js to function component  #20193

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

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

[NikkiWines]

oh hmm looks like we need a BZ member to issue payment to @aimane-chnaif for reviewing #20193 - asking about the appropriate label now

[NikkiWines]

@greg-schroeder kindly volunteered to help out here 🙇

[greg-schroeder]

created a job https://www.upwork.com/jobs/~014cc74cd0980ecb9b

sent you an offer for reviewing this @aimane-chnaif

[greg-schroeder]

Pending acceptance and then we can close