-
-
Notifications
You must be signed in to change notification settings - Fork 195
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
Stripe prices #8424
base: stripe-prices-refactor
Are you sure you want to change the base?
Stripe prices #8424
Conversation
479c599
to
3604552
Compare
3604552
to
d1e9029
Compare
d1e9029
to
a030c3c
Compare
@dracos it has been suggested that you might be the best places to review this. There are a lot of additional changes in the linked PRs which really someone should review but not sure if you've really got time to do that? |
Sure, I can take a look at them all |
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.
Looks good, couple of queries is all :)
7ada4b3
to
0994a53
Compare
Using Prices provides more flexibility and will allow us to change the which prices are used on both current and future subscriptions.
Run via: `rails subscriptions:migrate_price OLD_PRICE=pro NEW_PRICE=price_123`
Relevant issue(s)
https://github.com/mysociety/whatdotheyknow-private/issues/328
Depends on #8405
Depends on #8407
What does this do?
Switches from Stripe Plans to Prices allowing us to change subscriptions and update Pro our pricing model.