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
Add missing invoice attributes #425
Add missing invoice attributes #425
Conversation
@maartenvanvliet Would you mind updating this branch with master? Just merged in some dialyxir additions! |
lib/stripe/subscriptions/invoice.ex
Outdated
@@ -51,6 +53,7 @@ defmodule Stripe.Invoice do | |||
receipt_number: String.t() | nil, | |||
starting_balance: integer, | |||
statement_descriptor: String.t() | nil, | |||
status: String.t(), |
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.
String.t() | nil
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.
👍
@@ -33,10 +33,12 @@ defmodule Stripe.Invoice do | |||
currency: String.t(), | |||
customer: Stripe.id() | Stripe.Customer.t(), | |||
date: Stripe.timestamp(), | |||
default_source: String.t() | nil, |
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 like one can pass this field to update
and create
as well
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.
👍
a9f6ea0
to
7db6273
Compare
@maartenvanvliet Feel free to merge in master when you have a chance. Just to make sure we don't miss any dialyzer errors 👍. Then is is g2g! Nice work! |
I rebased on master 😀 |
Adds missing invoice attributes to the Invoice struct.
See: https://stripe.com/docs/api/invoices/object and https://raw.githubusercontent.com/stripe/openapi/master/openapi/spec3.yaml
Another attribute,
payment_intent
is also sent by Stripe, this is not documented so I've not added it.