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

WCPay Subscriptions renewals failed to renew paid with India-issued credit card #5311

Open
shendy-a8c opened this issue Dec 19, 2022 · 4 comments
Labels
category: core WC Payments core related issues, where it’s obvious. component: wcpay subscriptions Issues related to Stripe Billing Subscriptions focus: subscriptions priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. type: bug The issue is a confirmed bug.

Comments

@shendy-a8c
Copy link
Contributor

shendy-a8c commented Dec 19, 2022

Describe the bug

When purchasing WCPay Subscriptions product with India-issued credit card, renewals will fail to renew. I think this is because we haven't handled webhook event from Stripe that pass the mandate.

Example: https://dashboard.stripe.com/test/connect/accounts/acct_1MEEzXCMtaqOTQoE/subscriptions/sub_1MF3tqCMtaqOTQoEGSIYdqyx

To Reproduce

  1. Make sure WooCommerce Subscriptions is disabled.
  2. Create a WCPay Subscriptions product that renew daily and price > ₹15k.
  3. Set currency to INR at wp-admin > WooCommerce > Settings > Multi-currency.
  4. View the product in INR by appending ?currency=INR at product page or shop page.
  5. Purchase that WCPay Subscription product with 4000003560000123.
  6. Wait for 1 day.

Test with this card (not yet up on Stripe's docs, taken from p1667867044371529/1666705950.654039-slack-C9976E5MJ) and product price:

  • 4000003560000123, < ₹15k. This card simulates a mandate setup. Renewals should be successful.
  • 4000003560000297, any price. This card simulates customer receives predebit notif but cancel the mandate. Renewal should expectedly fail. Subscription should be on-hold.
  • 4000003560000248, any price. This card simulates issuing bank fails to send predebit notif. Renewal should expectedly fail. Subscription should be on-hold.
  • 4000003560000263, any price. This card simulates a mandate cancelled by user. Renewal should expectedly fail.

Actual behavior

Renewal will fail.

Expected behavior

Renewal should work.

Additional context

#3011 is the related PR that fixes the problem for WooCommerce Subscriptions product

@shendy-a8c shendy-a8c added type: bug The issue is a confirmed bug. component: wcpay subscriptions Issues related to Stripe Billing Subscriptions needs triage Manually put issue into triage process. labels Dec 19, 2022
@shendy-a8c
Copy link
Contributor Author

@haszari This issue needs triaging.

@haszari haszari added component: customer multi-currency Issues related to customer multi-currency project priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. and removed needs triage Manually put issue into triage process. labels Dec 19, 2022
@haszari
Copy link
Contributor

haszari commented Dec 19, 2022

Marking as low priority for now, on the assumption that there are very few merchants affected:

  • WCPay Subscriptions is US-only (though can be used with multicurrency)
  • Multi-currency enabled and INR enabled
  • Subscribers/customers paying in INR

We should fix this, especially if/when we roll out WCPay Subscriptions more widely. If we get reports from merchants affected we can also raise the priority.

@jessy-p jessy-p added the category: core WC Payments core related issues, where it’s obvious. label Dec 30, 2022
@haszari haszari added needs prioritisation Triage finished and issues are ready for the following processing. and removed needs prioritisation Triage finished and issues are ready for the following processing. labels Jan 11, 2023
@jessy-p
Copy link
Contributor

jessy-p commented Feb 2, 2023

@Automattic/helix

@pierorocca pierorocca removed the component: customer multi-currency Issues related to customer multi-currency project label Jan 18, 2024
@pierorocca
Copy link
Contributor

Noting that I removed the multi-currency label since the issue isn't related to multi-currency, only that changing the currency to INR allows the mandate issue on subscriptions, US only, to be exposed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: core WC Payments core related issues, where it’s obvious. component: wcpay subscriptions Issues related to Stripe Billing Subscriptions focus: subscriptions priority: low The issue/PR is low priority—not many people are affected or there’s a workaround, etc. type: bug The issue is a confirmed bug.
Projects
None yet
Development

No branches or pull requests

4 participants