Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[$250] [Payment card / Subscription] "Change payment card" button doesn’t work after using 3DS flow to add GBP payment card #46146

Closed
blimpich opened this issue Jul 24, 2024 · 36 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor

Comments

@blimpich
Copy link
Contributor

blimpich commented Jul 24, 2024

Problem

After adding a payment card using the 3DS flow we can't change the payment card.

Screen.Recording.2024-07-24.at.2.38.12.PM.mov

Solution

Figure out why and fix it.

Note:

You will need to use an internal Expensify engineer's ngrok credentials in order to test this. If you'd like to work on this please message @blimpich and I will provide you with instructions on how to use my local backend via ngrok. One of the catches with this is that my ngrok will only work when I'm online, so if you aren't able to be online during some portion of PST 9am-5pm hours then you might not want to work on this.

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~016cee3d2a1a851762
  • Upwork Job ID: 1819499472402209186
  • Last Price Increase: 2024-08-02
  • Automatic offers:
    • nkdengineer | Contributor | 103423143
Issue OwnerCurrent Issue Owner: @muttmuure
@blimpich blimpich added Engineering Daily KSv2 Internal Requires API changes or must be handled by Expensify staff Bug Something is broken. Auto assigns a BugZero manager. Hot Pick Ready for an engineer to pick up and run with labels Jul 24, 2024
Copy link

melvin-bot bot commented Jul 24, 2024

Triggered auto assignment to @muttmuure (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@blimpich
Copy link
Contributor Author

This really should be external I think, but non-Expensify engineers can't actually repro this issue until #46144 is completed, since the full 3DS flow isn't available to them locally (link). So marking this as internal hot pick for now.

@blimpich blimpich moved this to HOT PICKS in [#whatsnext] #wave-collect Jul 25, 2024
@melvin-bot melvin-bot bot added the Overdue label Jul 29, 2024
@trjExpensify
Copy link
Contributor

Could we send it external and let them connect to someone's VM or something like we've done before?

@blimpich
Copy link
Contributor Author

Even with access to an engineer's ngrok a external contributor won't be able to repro this. I'm working on this which I think should make it possible for external engineers to repro this with a Expensify engineer's ngrok credentials.

@muttmuure
Copy link
Contributor

Held on solution in linked issue

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Jul 30, 2024
Copy link

melvin-bot bot commented Aug 2, 2024

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

@blimpich
Copy link
Contributor Author

blimpich commented Aug 2, 2024

I think I've figured out a way to make this work with my local VM, so that contributors can just use my ngrok. I was having difficulty with contributors using flows that used Web-Secure but I think I have this working now. So lets mark this as external and see if anyone can fix it.

@blimpich blimpich added External Added to denote the issue can be worked on by a contributor and removed Internal Requires API changes or must be handled by Expensify staff Hot Pick Ready for an engineer to pick up and run with labels Aug 2, 2024
Copy link

melvin-bot bot commented Aug 2, 2024

Job added to Upwork: https://www.upwork.com/jobs/~016cee3d2a1a851762

@melvin-bot melvin-bot bot changed the title [Payment card / Subscription] "Change payment card" button doesn’t work after using 3DS flow to add GBP payment card [$250] [Payment card / Subscription] "Change payment card" button doesn’t work after using 3DS flow to add GBP payment card Aug 2, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Aug 2, 2024
Copy link

melvin-bot bot commented Aug 2, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @hungvu193 (External)

@melvin-bot melvin-bot bot removed the Overdue label Aug 2, 2024
@hungvu193
Copy link
Contributor

waiting for proposals here

@nkdengineer
Copy link
Contributor

You will need to use an internal Expensify engineer's ngrok credentials in order to test this

@blimpich I'd like to work on this issue. Can you share your ngrok then I can test and find a solution, thanks.

@blimpich
Copy link
Contributor Author

blimpich commented Aug 5, 2024

DM'ed @nkdengineer with credentials on Slack.

Copy link

melvin-bot bot commented Aug 21, 2024

@hungvu193 @muttmuure @nkdengineer this issue is now 4 weeks old, please consider:

  • Finding a contributor to fix the bug
  • Closing the issue if BZ has been unable to add the issue to a VIP or Wave project
  • If you have any questions, don't hesitate to start a discussion in #expensify-open-source

Thanks!

@nkdengineer
Copy link
Contributor

I'm working on this issue now

@hungvu193
Copy link
Contributor

Still working on the PR.

@nkdengineer
Copy link
Contributor

Currently, to test this PR, I need the Blimpich environment, and we need to be online at the same time, but we're in different regions. Will try to test it tonight.

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Aug 26, 2024
Copy link

melvin-bot bot commented Aug 28, 2024

Triggered auto assignment to @lakchote, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@blimpich blimpich assigned blimpich and unassigned lakchote Aug 28, 2024
@blimpich
Copy link
Contributor Author

Not sure why I wasn't assigned to this before, ignore the ping @lakchote

@hungvu193
Copy link
Contributor

This issue was deployed to prod 8 days ago, it's now ready for payment @muttmuure

@blimpich blimpich added Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 and removed Reviewing Has a PR in review Weekly KSv2 labels Sep 9, 2024
@melvin-bot melvin-bot bot added the Overdue label Sep 12, 2024
@muttmuure
Copy link
Contributor

$250 - @hungvu193
$250 - @nkdengineer

@melvin-bot melvin-bot bot added Overdue Daily KSv2 and removed Overdue Daily KSv2 labels Sep 13, 2024
@garrettmknight
Copy link
Contributor

$250 approved for @hungvu193

@muttmuure
Copy link
Contributor

@nkdengineer paid

@melvin-bot melvin-bot bot removed the Overdue label Sep 19, 2024
@github-project-automation github-project-automation bot moved this from Polish to Done in [#whatsnext] #wave-collect Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering External Added to denote the issue can be worked on by a contributor
Projects
No open projects
Status: Done
Development

No branches or pull requests

7 participants