Skip to content

[Awaiting Payment 26th August] [$250] [Payment card / Subscription] Make change billing card currency form wait until request has finished before closing #45645

Closed

Description

@blimpich

Problem

When changing the billing currency on the subscription page, the panel closes immediately after clicking "save". This is a problem because if the request has an error we should show that error in the panel, but the panel will have already closed.

Screen.Recording.2024-07-29.at.4.54.04.PM.mov

Solution

We should have this panel behave the same way that the add payment card panel behaves. It should wait for the request to finish and show a loading indicator while we wait.

cc: @dominictb

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01bcbde08595f89c2b
  • Upwork Job ID: 1818075810728864794
  • Last Price Increase: 2024-07-30
  • Automatic offers:
    • nkdengineer | Contributor | 103346635
Issue OwnerCurrent Issue Owner: @rushatgabhane

Activity

  1. self-assigned this
    on Jul 17, 2024
  2. blimpich commented on Jul 17, 2024

    @blimpich
    ContributorAuthor

    Holding and not adding the external label until necessary backend changes are done

  3. melvin-bot commented on Jul 23, 2024

    @melvin-bot

    @blimpich Huh... This is 4 days overdue. Who can take care of this?

  4. blimpich commented on Jul 23, 2024

    @blimpich
    ContributorAuthor

    Almost there, backend changes are still in review.

  5. melvin-bot commented on Jul 29, 2024

    @melvin-bot

    @blimpich Huh... This is 4 days overdue. Who can take care of this?

  6. changed the title [-][Payment card / Subscription][HOLD #45124] Make change billing card currency form wait until request has finished before closing[/-] [+][Payment card / Subscription] Make change billing card currency form wait until request has finished before closing[/+] on Jul 29, 2024
  7. added
    ExternalAdded to denote the issue can be worked on by a contributor
    NewFeatureSomething to build that is a new item.
    on Jul 30, 2024
  8. 32 remaining items

  9. garrettmknight commented on Aug 20, 2024

    @garrettmknight
    Contributor

    @isabelastisser looks like the PR is on prod btw.

  10. isabelastisser commented on Aug 20, 2024

    @isabelastisser
    Contributor

    Thanks for the heads up, @garrettmknight! So we're waiting for the PR to merge.

  11. trjExpensify commented on Aug 26, 2024

    @trjExpensify
    Contributor

    PR hit prod on the 19th August. #46635

  12. added
    Awaiting PaymentAuto-added when associated PR is deployed to production
    and removed
    ReviewingHas a PR in review
    on Aug 26, 2024
  13. changed the title [-][$250] [Payment card / Subscription] Make change billing card currency form wait until request has finished before closing[/-] [+][Awaiting Payment 26th August] [$250] [Payment card / Subscription] Make change billing card currency form wait until request has finished before closing[/+] on Aug 26, 2024
  14. isabelastisser commented on Aug 27, 2024

    @isabelastisser
    Contributor

    Payment summary:

    @rushatgabhane $250 / C+ review / NewDot payment PENDING
    @nkdengineer $250 / Contributor / Paid in Upwork

  15. JmillsExpensify commented on Aug 27, 2024

    @JmillsExpensify
    Contributor

    $250 approved for @rushatgabhane

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 contributorNewFeatureSomething to build that is a new item.

Type

No type

Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions