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

[WIP] sync_subscription_from_stripe_data: use sync_plan #20

Open
wants to merge 2 commits into
base: next
Choose a base branch
from

Conversation

blueyed
Copy link

@blueyed blueyed commented Oct 20, 2017

This ensures that the plan exists locally, which might not be the case
when it has been deleted in Django, but still exists on Stripe.

TODO:

  • test

@blueyed blueyed force-pushed the sync_subscription_from_stripe_data-sync_plan branch from f0cf9cc to 9e1d98e Compare October 26, 2017 13:28
@blueyed
Copy link
Author

blueyed commented Oct 26, 2017

test_customer_subscription_deleted needs to be fixed, have it in a stash.

paltman and others added 2 commits October 26, 2017 15:28
This ensures that the plan exists locally, which might not be the case
when it has been deleted in Django, but still exists on Stripe.
@blueyed blueyed force-pushed the sync_subscription_from_stripe_data-sync_plan branch from 9e1d98e to 11b4239 Compare October 26, 2017 13:29
@blueyed blueyed self-assigned this Oct 26, 2017
@blueyed blueyed changed the title sync_subscription_from_stripe_data: use sync_plan [WIP] sync_subscription_from_stripe_data: use sync_plan Oct 26, 2017
@blueyed blueyed removed their assignment Sep 27, 2019
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 this pull request may close these issues.

2 participants