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.

Sign up for GitHub

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

Jump to bottom

Stripe payment field does not hide when 100% discount coupon applied #1171

Closed
FVP56 opened this issue Nov 1, 2019 · 1 comment
Closed

Stripe payment field does not hide when 100% discount coupon applied #1171

FVP56 opened this issue Nov 1, 2019 · 1 comment
Labels
bug

Comments

@FVP56
Copy link

FVP56 commented Nov 1, 2019

v191022: In reference to s2Member Pro Feature Enhancement you made the credit card field inline. However when you made the Stripe payment fields inline with the s2Member form, you apparently forgot to hide those when applying any coupon for 100% discount. A workaround or fix would be much appreciated since we have many forms that now present the required credit card field after applying a coupon for 100% discount which is causing much confusion and frustration for our members.
@iaeon
Copy link

iaeon commented Nov 12, 2019

I can confirm that s2Member Pro v191022 with Stripe suffers from the problem as described above. A 100% discount code is recognised but the 'Billing Method' section is NOT hidden as it should be therefore card payment is still required. S2Member / S2MemberPro is effectively broken for the purposes of 100% discounts until this issue is addresses. Also causing considerable consternation and confusion so please fix urgently.

@clavaque clavaque added the bug label Nov 14, 2019
@clavaque clavaque mentioned this issue Feb 4, 2020
Closed
@clavaque clavaque closed this as completed Mar 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@clavaque @iaeon @FVP56