Skip to content

UPE: Missing equivalent to the wc_stripe_force_save_source hook #3502

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

Open
sebcode opened this issue Oct 4, 2024 · 2 comments · May be fixed by #4243
Open

UPE: Missing equivalent to the wc_stripe_force_save_source hook #3502

sebcode opened this issue Oct 4, 2024 · 2 comments · May be fixed by #4243
Assignees
Labels
priority: medium The issue/PR is medium priority—it affects lots of customers substantially, but not critically.

Comments

@sebcode
Copy link

sebcode commented Oct 4, 2024

Before switching to the new checkout experience we used add_filter('wc_stripe_force_save_source', '__return_true'); which attaches the stripe payment method to the stripe customer for our subscription service (we're not using the official Woocommerce Subscriptions plugin). There doesn't seem to be an equivalent hook for the new checkout experience.

For now as a hacky workaround I'm defining the functions wcs_order_contains_subscription, wcs_is_subscription, wcs_order_contains_renewal to make WC_Stripe_Subscriptions_Utilities_Trait::has_subscription() return true for certain orders. This causes the stripe plugin to set setup_future_usage:off_session for the payment intent which is what we want for our use case. It would be great if there would be a cleaner way to do that.

Copy link

github-actions bot commented Mar 4, 2025

Hi,
This issue has gone 150 days (5 months) without any activity. This means it is time for a check-in to make sure it is still relevant. If you are still experiencing this issue with the latest version, you can help the project by responding to confirm the problem and by providing any updated reproduction steps.
Thanks for helping out.

@Mayisha Mayisha added priority: medium The issue/PR is medium priority—it affects lots of customers substantially, but not critically. and removed status: needs confirmation labels Mar 20, 2025
@Mayisha
Copy link
Contributor

Mayisha commented Mar 20, 2025

@Mayisha Mayisha self-assigned this Apr 16, 2025
@Mayisha Mayisha linked a pull request Apr 18, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: medium The issue/PR is medium priority—it affects lots of customers substantially, but not critically.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants