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

Store payment method and source on subscriptions #127

Merged
merged 4 commits into from
Jun 18, 2020

Conversation

aldesantis
Copy link
Member

@aldesantis aldesantis commented Jun 17, 2020

Closes #116.

Implements the ability to have different payment methods and sources on each subscription. The payment information is copied from the order that originated the subscription, when possible.

Because the payment method is also stored, this PR also opens up the possibility of charging subscriptions via non-credit-card payment methods such as checks or wire transfers. All you have to do is set the right payment_method on the subscription and not set the payment_source.

@aldesantis aldesantis self-assigned this Jun 17, 2020
@aldesantis aldesantis changed the title Aldesantis/subscription payment source Store payment source on subscriptions Jun 17, 2020
@aldesantis aldesantis added the enhancement Improves an existing feature. label Jun 17, 2020
@aldesantis aldesantis changed the title Store payment source on subscriptions [WIP] Store payment source on subscriptions Jun 17, 2020
@aldesantis aldesantis force-pushed the aldesantis/subscription-payment-source branch 3 times, most recently from 9331b3f to 5a7da65 Compare June 18, 2020 13:40
@aldesantis aldesantis changed the title [WIP] Store payment source on subscriptions Store payment method and source on subscriptions Jun 18, 2020
This allows each subscription to have its own payment method and
source. It also allows having subscriptions with payment methods
that aren't credit cards, which is useful if you want e.g. to allow
users to pay by check/wire transfer.
@aldesantis aldesantis force-pushed the aldesantis/subscription-payment-source branch 8 times, most recently from a0530ae to 97ef706 Compare June 18, 2020 14:49
@aldesantis aldesantis force-pushed the aldesantis/subscription-payment-source branch from 97ef706 to 626dd1c Compare June 18, 2020 14:52
@aldesantis aldesantis merged commit 2dc2446 into master Jun 18, 2020
@aldesantis aldesantis deleted the aldesantis/subscription-payment-source branch June 18, 2020 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improves an existing feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Store payment details on subscription
1 participant