-
Notifications
You must be signed in to change notification settings - Fork 215
Including support for physical free trial subscription products with ECE #4816
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
base: develop
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wjrosa do you see the following error for a free trial subscription with ECE? I see this error when I click the Google Pay button on both product and checkout pages. Works fine for simple products and subscriptions without a free trial for the same address.
I can continue and pay, but was wondering if this will be an issue in live mode.
daledupreez
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am seeing the same issue as @Mayisha when computing shipping options in both Google Pay and Apple Pay.
Co-authored-by: daledupreez <dale.du.preez@automattic.com>
|
I did some deep debugging today and found out the issue is due the subscriptions extension. We have this filter there: It does what the comment says. If I comment out this filter, it works just fine. That must be why we blocked physical free trials in the first place. I will ask the team about it |
|
Asked the Chronos team about it in p1764795994520959-slack-C0866MKN6H1 |
|
@Mayisha @daledupreez I decided to remove the filter above in 860684e. Per the discussion in p1764795994520959-slack-C0866MKN6H1, I believe it makes sense. I did some testing, and things are working fine for me. Let me know what you think |
Fixes STRIPE-715
Changes proposed in this Pull Request:
Our current implementation of the Express Checkout Element does not support free-trial subscriptions that require shipping. I am adding support for that here.
We are basically removing the constraint for physical products from the backend. Until now, we could not find reasons why this could break (and was prevented).
Testing instructions
add/include-support-for-physical-free-trials)Changelog entry
Changelog Entry Comment
Comment
Post merge