Skip to content

[$250] Migrate DragAndDrop to function component #16142

Closed

Description

@marcaaron

Class Component Migration

Filenames

  • src/components/DragAndDrop/index.js
    • type: React.Component
    • has state values: false
    • has refs: false
    • has context: false
    • lifecycle methods: componentDidMount,componentWillUnmount,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/~0175ef7de3aea53b93
  • Upwork Job ID: 1691269822034804736
  • Last Price Increase: 2023-08-15

Activity

  1. locked and limited conversation to collaborators on Mar 20, 2023
  2. unlocked this conversation on Mar 21, 2023
  3. changed the title [-][HOLD] Migrate index.js to function component[/-] [+][HOLD][$250] Migrate index.js to function component[/+] on Apr 13, 2023
  4. muxriddinmuqimov77 commented on Jun 6, 2023

    @muxriddinmuqimov77

    I'd like to work on this issue

  5. dummy-1111 commented on Jun 6, 2023

    @dummy-1111
    Contributor

    I'd love to work on this.

  6. vdem0115 commented on Jul 7, 2023

    @vdem0115

    I'd like to work on this issue

  7. olexyt commented on Jul 7, 2023

    @olexyt
    Contributor

    I can work on this issue.

  8. mejed-alkoutaini commented on Jul 11, 2023

    @mejed-alkoutaini

    I'm able to work on this if needed!

  9. mkhutornyi commented on Jul 11, 2023

    @mkhutornyi
    Contributor

    I'd like to work on this.

  10. KrAbhas commented on Jul 19, 2023

    @KrAbhas
    Contributor

    I would like to work on this

  11. jczekalski commented on Jul 19, 2023

    @jczekalski
    Contributor

    Hey! I’m from Software Mansion, an expert agency, and I’d like to work on this issue.

  12. 24 remaining items

  13. added
    ExternalAdded to denote the issue can be worked on by a contributor
    on Aug 15, 2023
  14. melvin-bot commented on Aug 15, 2023

    @melvin-bot
  15. added
    Help WantedApply this label when an issue is open to proposals by contributors
    on Aug 15, 2023
  16. melvin-bot commented on Aug 15, 2023

    @melvin-bot

    Triggered auto assignment to @bfitzexpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

  17. melvin-bot commented on Aug 15, 2023

    @melvin-bot

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

  18. removed
    Help WantedApply this label when an issue is open to proposals by contributors
    on Aug 15, 2023
  19. roryabraham commented on Aug 15, 2023

    @roryabraham
    Contributor

    This is already done, we just need to pay @allroundexperts for reviewing. There were a couple regressions caused by this (my bad), so I think that means we should pay 50%

  20. changed the title [-][HOLD for payment 2023-08-08] [$250] Migrate DragAndDrop to function component[/-] [+][$250] Migrate DragAndDrop to function component[/+] on Aug 15, 2023
  21. allroundexperts commented on Aug 15, 2023

    @allroundexperts
    Contributor

    @roryabraham I think it was decided to not apply the penalty here. ref

  22. bfitzexpensify commented on Aug 17, 2023

    @bfitzexpensify
    Contributor

    Cool. @allroundexperts eligible for internal PR review payment for $1000, to be made via manual request.

  23. JmillsExpensify commented on Aug 24, 2023

    @JmillsExpensify
    Contributor

    Reviewed the details for @allroundexperts. $1,000 approved for payment in NewDot based on the BZ summary above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionDailyKSv2EngineeringExternalAdded to denote the issue can be worked on by a contributorImprovementItem broken or needs improvement.

Type

No type

Projects

No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions