Comments (24)
Critical is a when you couldn't do anything to proceed, e.g. fatal error or you can't proceed to checkout for whatever reason. The cards on Stripe still exist and customer can add it, it just missing on the new token table.
This can be a tricky to support as customer cards never stored permanently (just transient) in WP before, that means it should checks each customer, fetches cards from Stripe, and store it to new token table. What I could think of, for the easiest way, is when my payment methods empty and there's Stripe customer_id in user's meta we check cards for that customer_id and store any result it found to the new token table, what do you think?
/cc @mikejolley
from woocommerce-gateway-stripe.
Hi I'm just going through a rather long over due upgrade from gateway 2.0.3 to 3.0.2 and had this very issue.
As @MrFent pointed out if you go to the checkout page your card tokens are pulled in and saved but viewing the My Account Page does not.
The removal of woocommerce_get_customer_payment_tokens doesn't fix this issue for me as I think @gedex saw.
I have a work around that works for me as pull request #34 but would love your feedback on this.
from woocommerce-gateway-stripe.
Another ticket:
https://woothemes.zendesk.com/agent/tickets/432487
from woocommerce-gateway-stripe.
Updated label to high from critical as it doesn't block functionality.
from woocommerce-gateway-stripe.
@gedex I feel it does block functionality as customers can not longer access their saved cards 🎱 :)
from woocommerce-gateway-stripe.
I tested this with the commits to fix the issue: [Fixed issue where no param being passed to Stripe in pay order page. #27]
And it fixed the issue with old saved cards not appearing on My Account page, however it only worked if I first added a product to my cart and then visit the checkout page.
Is there a way to make it so it'll work right off the bat by going to the my account/Payment Methods page without having to go to the checkout page first?
from woocommerce-gateway-stripe.
@MrFent sorry that PR should fixes #23 instead of #19. Reopened.
from woocommerce-gateway-stripe.
Thank you, and regarding my previous comment... I just realized that even without applying the fix for #27, the saved cards will still appear once I visit the checkout page first.
from woocommerce-gateway-stripe.
@gedex I don't think #24 fixes this issue.
from woocommerce-gateway-stripe.
Hey @dwainm @gedex what do you think about PR #34 , thanks :)
from woocommerce-gateway-stripe.
@ritchiem @spraveenitpro not sure if the PR will fix the issue. My only concern with the PR is that it fetches cards even though selected payment method is not Stripe.
To fix this issue, it should performs background updater and check any cards it found before to new token table.
from woocommerce-gateway-stripe.
Sorry for slow reply.
There is no selected payment method when the My Account page loads the consumer payment tokens. Or at least the changed method in the PR was never called with 'stripe' == $gateway_id when I was trying to debug this section on the My Account page.
A background updater would be a way of helping this however, it would not solve the issue as you still need to have an on demand migration to guard against the situation where the customer visits their My Account page before the background updater has reached their account.
As a result I personally prefer on demand migrations at time of access where possible as that way you never miss a migration situation. If data consistency is important for some other system access then have an additional updating process is justified. Taking the site down to do a migration run is never acceptable.
If the concern is the API call load on Stripe then perhaps the best solution is to check $tokens for any existing saved stripe tokens thus preventing the need to reload.
Given this is supposed to run on every cart checkout to ensure we have the latest saved cards I don't see a major issue with loading it for the My Account Saved Cards page as consistency of experience is what matters for a good user experience.
from woocommerce-gateway-stripe.
Any progress on this issue?
Updating the latest WC version is becoming more and more urgent for my website. But I do not want to do this until the saved card issue is resolved...
from woocommerce-gateway-stripe.
Olà folks,
I just came across the issue in:
https://woothemes.zendesk.com/agent/tickets/488645
It seems as if there is a WIP fix here: #43 - forgive my laymen's view if I am wrong.
Any chances we could push this forward to 🏁? 💌
from woocommerce-gateway-stripe.
Possibly related? 506271-zd-woothemes
from woocommerce-gateway-stripe.
@allendav not related. This issue is more about adding data migration when people update from WC 2.5.5 to 2.6.x. In WC 2.5.5, payment token was not born yet so saved cards were saved in transients (or maybe user meta?). WC 2.6.x and Stripe 3.0 both use payment token API which uses custom tables. People upgraded won't see the saved cards that previously stored in transients (though it's not completely missing as the real card data is in Stripe).
from woocommerce-gateway-stripe.
Any update on this? We just upgraded from WC 2.5.5 and are running into this issue. We use WooCommerce subscriptions and it's quite a big issue for existing subscribers to go to their account and not see their credit cards!
from woocommerce-gateway-stripe.
@spraveenitpro - has this issue come up again since reported?
I'd like to see us work on a way to remedy this for affected merchants (would you be open to testing with us, @numenon ?), without including that code in the Stripe integration itself.
from woocommerce-gateway-stripe.
On further reading, I don't actually mind having a small piece of migration code in the extension, if it helps merchants as the quickest route.
@spraveenitpro - has this come up again since I posted last?
Assigning to @gedex, for attention in the next few weeks, after we wrap up a collection of shipping fixes.
from woocommerce-gateway-stripe.
@mattyza it has not come up since then :)
from woocommerce-gateway-stripe.
Thanks @spraveenitpro.
@gedex - I like the idea of exploring what this would look like, as a migration script within the extension which shows itself only if necessary.
from woocommerce-gateway-stripe.
@mattyza With WC 3.1 as the latest stable, not sure if this still worth to add into our Stripe extension. IIRC, this only affects WC 2.5.x, but I need to checkout the old branch of WC and Stripe to see if I could still reproduce. If we still pursue this, I'd prefer a small one-time plugin that does the migration.
from woocommerce-gateway-stripe.
@gedex - we support 2.6.x and higher. If this affects a 2.6.x WooCommerce store, we should make sure we take care of this.
A one-time plugin could be fine, although I'd prefer the customer not have to install something else, just to fix an issue we caused. Thus, I'd love to have this as part of the extension, which only shows when it needs to, and then hides itself.
from woocommerce-gateway-stripe.
Closing this as too much time has passed and merchants would have already updated and refreshed the cards by now so is no longer relevant. We don't have any new complaints on this with current versions. Also this seems to only affect merchants with WC 2.5.x.
cc @mattyza if you still think this is needed please reopen.
from woocommerce-gateway-stripe.
Related Issues (20)
- Refunds via Giropay not available after deprecation
- Deprecate Key Sharing - Add re-connect promotional surface
- Minor text updates to webhook settings
- Clear the webhook status when configuring a new one
- Creating a setup intent with a new payment method fails when using the Legacy checkout experience
- Non-existent billing field throwing JS errors on console
- Flag accounts that are connected via the oauth flow and use it to display connected status
- Update the `WC_Stripe_Connect_API` class to support test mode requests
- Webhooks should be configured automatically when connecting via OAuth
- Enable express checkout capability when using Yith Multi-Vendor plugin with shipping enabled
- Missing Payment Method: Paypal HOT 1
- Missing Payment Method: Twint HOT 1
- Duplicate charges when placing order (if "Link By Stripe" was previously connected) HOT 3
- Failing refund e2e test
- Merchant Message Visible on Checkout Page HOT 1
- [HC] - Disabling the CC option makes Stripe APMs show up first, not respecting the order set in the overall WooCommerce Payment settings
- Updating Subscription's payment method to a 3DS card results in an error HOT 1
- Display a notice when the empty state Connect API oauth URL is missing
- Update the empty state connect flow to support connecting test mode
- Why is stylesheet with ID 'wc-stripe-blocks-checkout-style' enqueued everywhere?
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from woocommerce-gateway-stripe.