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
api version upgrade #181
api version upgrade #181
Conversation
1125160
to
6754bc0
Compare
tests/utils_invoices.py
Outdated
item = stripe.InvoiceItem.create( | ||
amount=random.randint(1, 10000), | ||
currency="usd", | ||
customer=customer_id, | ||
description="Comfortable cotton t-shirt {}".format(now_value), | ||
metadata=metadata_value, | ||
discountable=True, | ||
subscription_item=None, | ||
tax_rates=[], # TODO enter the child attributes | ||
stripe_version = '2020-08-27' |
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.
Indent these like you have below
tests/utils_invoices.py
Outdated
# custom_fields={ | ||
# 'name': 'CustomName', | ||
# 'value': 'CustomValue', | ||
# }, |
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.
Let's delete these lines
tests/test_all_fields.py
Outdated
# Fields which are deprecated and no longer will come in the response of api | ||
deprecated_fields = { | ||
'payment_intents': { | ||
'charges' | ||
} | ||
} | ||
|
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.
Should this just be included in FIELDS_TO_NOT_CHECK
?
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.
Tests could use some clean up / formatting, but the changes look fine to me
Description of change
Manual QA steps
Risks
Rollback steps