-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Comments
Triggered auto assignment to @muttmuure ( |
Could we send it external and let them connect to someone's VM or something like we've done before? |
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. |
Held on solution in linked issue |
@muttmuure Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
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. |
Job added to Upwork: https://www.upwork.com/jobs/~016cee3d2a1a851762 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @hungvu193 ( |
waiting for proposals here |
@blimpich I'd like to work on this issue. Can you share your ngrok then I can test and find a solution, thanks. |
DM'ed @nkdengineer with credentials on Slack. |
@hungvu193 @muttmuure @nkdengineer this issue is now 4 weeks old, please consider:
Thanks! |
I'm working on this issue now |
Still working on the PR. |
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. |
Triggered auto assignment to @lakchote, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
Not sure why I wasn't assigned to this before, ignore the ping @lakchote |
This issue was deployed to prod 8 days ago, it's now ready for payment @muttmuure |
$250 - @hungvu193 |
$250 approved for @hungvu193 |
@nkdengineer paid |
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
Issue Owner
Current Issue Owner: @muttmuureThe text was updated successfully, but these errors were encountered: