Skip to content

[$125] [Form Provider Refactor] ACHContractStep #30314

Closed

Description

@luacmartins

Coming from #25397 (comment), we created a new Form provider that works with functional components and we need to refactor all usages of the old Form component.

This issue is specific to refactoring src/pages/ReimbursementAccount/ACHContractStep.js

cc @kowczarz @cdOut

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01cc4521b1d4b152cc
  • Upwork Job ID: 1716963452301692928
  • Last Price Increase: 2023-10-26
Issue OwnerCurrent Issue Owner: @cdOut

Activity

  1. added
    InternalRequires API changes or must be handled by Expensify staff
    BugSomething is broken. Auto assigns a BugZero manager.
    on Oct 24, 2023
  2. self-assigned this
    on Oct 24, 2023
  3. melvin-bot commented on Oct 24, 2023

    @melvin-bot
  4. melvin-bot commented on Oct 24, 2023

    @melvin-bot

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

  5. melvin-bot commented on Oct 24, 2023

    @melvin-bot
  6. melvin-bot commented on Oct 24, 2023

    @melvin-bot

    Triggered auto assignment to Contributor Plus for review of internal employee PR - @ntdiary (Internal)

  7. luacmartins commented on Oct 25, 2023

    @luacmartins
    ContributorAuthor

    @cdOut will you be working on this issue too?

  8. cdOut commented on Oct 26, 2023

    @cdOut
    Contributor

    @luacmartins yes, I will be working on this issue.

  9. added
    ExternalAdded to denote the issue can be worked on by a contributor
    and removed
    InternalRequires API changes or must be handled by Expensify staff
    on Oct 26, 2023
  10. 24 remaining items

  11. twisterdotcom commented on Nov 16, 2023

    @twisterdotcom
    Contributor

    Any update here @cdOut?

  12. twisterdotcom commented on Nov 16, 2023

    @twisterdotcom
    Contributor

    Ah wait, I see: #30665 (comment)

  13. ntdiary commented on Nov 17, 2023

    @ntdiary
    Contributor

    Ah wait, I see: #30665 (comment)

    @twisterdotcom, it looks like the refactoring has also been completed in another PR #30568. I noticed they modified the same code.

  14. cdOut commented on Nov 21, 2023

    @cdOut
    Contributor

    This issue was fixed in #30568, both flows were using a specific subform that had to be updated, so I had to update all the forms using it. This component is already refactored into FormProvider and is currently merged on main.

  15. twisterdotcom commented on Nov 21, 2023

    @twisterdotcom
    Contributor

    Looks like this caused some deployblockers: #30568 (comment). I don't think we can call this done yet.

  16. twisterdotcom commented on Dec 12, 2023

    @twisterdotcom
    Contributor

    Can we close this out now actually @cdOut and @luacmartins?

  17. cdOut commented on Dec 12, 2023

    @cdOut
    Contributor

    Yes, this is already merged and on main, as far as I'm concerned the deploy blockers were regarding different parts of form migration.

  18. luacmartins commented on Dec 12, 2023

    @luacmartins
    ContributorAuthor

    Yea, we're good to close this one. Thanks everyone!

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

Metadata

Metadata

Labels

BugSomething is broken. Auto assigns a BugZero manager.ExternalAdded to denote the issue can be worked on by a contributorReviewingHas a PR in reviewWeeklyKSv2

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions