[$250] Migrate ReportScreen.js to function component

[marcaaron]

Class Component Migration

Filenames

• src/pages/home/ReportScreen.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

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~0184e14afba16fa311
• Upwork Job ID: 1686461318953848833
• Last Price Increase: 2023-08-01
• Automatic offers:
• aimane-chnaif | Reviewer | 25887921

[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'd like to work on this issue

[dayana7204]

I am ready to work on this ticket. :)

[marcaaron]

@dayana7204 Kindly refrain from spamming all the issues that are on HOLD. Once they are off hold you will have an opportunity to apply.

[dayana7204]

Hi Marc, Really sorry for the inconvenience. Just wanted to get assigned the tasks once those are opened. Let me stop it now :) Regards, Dayana.

…

On Tue, Jun 27, 2023 at 4:08 PM Marc Glasser ***@***.***> wrote: @dayana7204 <https://github.com/dayana7204> Kindly refrain from spamming all the issues that are on HOLD. Once they are off hold you will have an opportunity to apply. — Reply to this email directly, view it on GitHub <#16255 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BATBWIWAM33KHWFAIYEM3P3XNM4THANCNFSM6AAAAAAWAY4MYI> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[rayane-d]

Can I work on this?

[to-ai]

I am able to work on this task.

[ghost]

Dibs

[pradeepmdk]

i will work on this

[SosenWiosen]

Hi, I'm from Software Mansion, an expert agency, and I'd like to work on this issue.

[melvin-bot]

Triggered auto assignment to @flaviadefaria (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/~0184e14afba16fa311

[melvin-bot]

📣 @aimane-chnaif 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job

[ghost]

Dibs

[ishpaul777]

Would love to take this

[hanzalahsamana]

The proposal aims to migrate the existing ReportScreen.js file from a class component to a function component with React hooks. This migration will enhance code readability, maintainability, and adhere to modern React best practices. The steps include refactoring the class component to a function component, removing unnecessary state and lifecycle methods, introducing React hooks for state management and side effects, handling props, and updating event handlers. The benefits of this migration include improved code readability, better performance, and easier future upgrades. The goal is to provide a more robust and efficient ReportScreen component for the application.

[melvin-bot]

📣 @hanzalahsamana! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

1. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
2. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
3. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
   
   Format:

Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

[Habben101]

Would love to work on this

[hanzalahsamana]

Your Expensify account email: hanzalahsamana789@gmail.com
Upwork Profile Link: https://www.upwork.com/freelancers/~019b0a755f7bda1754

[Muhammad-Saqib-980]

I would love to work on this.

[melvin-bot]

📣 @Muhammad-Saqib-980! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

1. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
2. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
3. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
   
   Format:

Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

[Muhammad-Saqib-980]

Contributor details
Your Expensify account email: saqibtariq980@gmail.com
Upwork Profile Link: https://www.upwork.com/freelancers/~010d8b5416650c0aa1

[melvin-bot]

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

[Re-Dei]

I would like to work on this.

[melvin-bot]

📣 @Re-Dei! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

1. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
2. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
3. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
   
   Format:

Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

[Re-Dei]

Contributor details
Your Expensify account email: hridaya.pradhan123@gmail.com
Upwork Profile Link: https://www.upwork.com/freelancers/~0169022423ba77b59f

[melvin-bot]

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

[alexxxwork]

I would love to work on this issue

[SosenWiosen]

@aimane-chnaif waiting for a review :)

[melvin-bot]

This issue has not been updated in over 15 days. @marcaaron, @SosenWiosen, @flaviadefaria, @aimane-chnaif eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

[aimane-chnaif]

PR was deployed to production on Aug 30

[blazejkustra]

@marcaaron Can we close this issue as PR was merged a while ago? 😄

[aimane-chnaif]

yes, we can close this issue

[flaviadefaria]

Closing this.