-
Notifications
You must be signed in to change notification settings - Fork 572
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
A few Billing changes #1758
A few Billing changes #1758
Conversation
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.
Can you also add the missing parameters to UpcomingInvoiceListLineItemOptions
?
a26afe1
to
f6a207e
Compare
Fixed, PTAL |
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.
One comment, LGTM otherwise.
f6a207e
to
aaeb5fe
Compare
Fixed, PTAL. |
aaeb5fe
to
9e4c969
Compare
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.
LGTM
Won't merge until stripe-java is ready so that we can triple-check the changes. |
* Add `Schedule` to `Subscription` * Add missing parameters to `UpcomingInvoiceOptions` and `UpcomingInvoiceListLineItemsOptions` * `Schedule` * `SubscriptionCancelAt` * `SubscriptionCancelNow` * Add missing properties and parameters for a `SubscriptionSchedule` phase: * `BillingThresholds` * `CollectionMethod` * `DefaultPaymentMethod` * `InvoiceSettings`
9e4c969
to
153dc3e
Compare
Fixes #1757 and other things I found:
A few Billing changes
Schedule
toSubscription
UpcomingInvoiceOptions
andUpcomingInvoiceListLineItemsOptions
Schedule
SubscriptionCancelAt
SubscriptionCancelNow
SubscriptionSchedule
phase:BillingThresholds
CollectionMethod
DefaultPaymentMethod
InvoiceSettings
r? @ob-stripe
cc @stripe/api-libraries