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

fix: show stripe credit card fields after update #4274

Closed
3 tasks done
mikejhale opened this issue Oct 8, 2019 · 0 comments · Fixed by #4275
Closed
3 tasks done

fix: show stripe credit card fields after update #4274

mikejhale opened this issue Oct 8, 2019 · 0 comments · Fixed by #4275
Assignees

Comments

@mikejhale
Copy link
Contributor

mikejhale commented Oct 8, 2019

Bug Report

User Story

As an admin, I want credit stripe credit card fields to show after updating so that I can continue to accept donations without having to change the donation flow on my site.

Current Behavior

After updating, Stripe Credit Card fields will not display on the donation form.

Expected Behavior

Stripe Credit Card fields show after update

Bug Type

  • This bug describes functionality that once worked as expected in version 2.5.4

Steps to Reproduce

This is a fairly edge case, but under the following circumstances Stripe fields won't show up on the page on the latest:

  1. Using the API Key method to connect
  2. Using Stripe Checkout modal on an older version of Core/Stripe addon.
  3. Update to 2.5.8 of core and 2.2.2 of Stripe Premium
  4. The credit card fields won't show up, no matter what.

Visuals

A detailed explanation and steps to reproduce are here:
https://screencast-o-matic.com/watch/cq6jYQu331

Related

Probably related to #4260

https://give.productboard.com/inbox/features/3056649/detail

Acceptance Criteria

  • Credit Card form fields are displayed after upgrade
  • Donation is successfully made using Stripe Credit Card
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants