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

Next major release changes #845

Merged
merged 7 commits into from
Aug 1, 2022
Merged

Next major release changes #845

merged 7 commits into from
Aug 1, 2022

Conversation

pakrym-stripe
Copy link
Contributor

@pakrym-stripe pakrym-stripe commented Jul 26, 2022

Changelog

4.0.0 - 2022-08-01

Breaking changes that arose during code generation of the library that we postponed for the next major version. For changes to the SDK, read more detailed description at https://github.com/stripe/stripe-python/wiki/Migration-guide-for-v4. For changes to the Stripe products, read more at https://stripe.com/docs/upgrades#2022-08-01.

"⚠️" symbol highlights breaking changes.

⚠️ Removed

  • Removed deprecated AlipayAccount, BitcoinReceiver, BitcoinTransaction, IssuerFraudRecord, Recipient, RecipientTransfer, and ThreeDSecure classes.
  • Removed deprecated Charge.update_dispute and Charge.close_dispute methods that were using legacy REST API endpoint. Prefer Dispute.modify and Dispute.close
  • Removed deprecated Card.details method and CardDetails resource. The REST API endpoint is not longer supported.
  • Removed the deprecated Source.source_transactions method. Prefer SubscriptionItem.list_source_transactions
  • Removed the deprecated SubscriptionItem.usage_record_summaries method. Prefer SubscriptionItem.list_usage_record_summaries
  • Removed the deprecated Charge.refund method. Prefer Refund.create

⚠️ Changed

  • To be consistent with other resource methods, ApplicationFee.refund returns an instance of ApplicationFee and doesn't mutate the instance of ApplicationFee.
  • To be consistent with other resource methods, the Customer.delete_discount no longer resets the discount property to None and returns the deleted discount instead. If you were relying on this behavior, reset the discount property manually:
  • The LineItem resource now inherits from StripeObject as it has no methods of it's own.
  • To be consistent with other resource methods, the Subscription.delete_discount returns an instance of deleted discount and doesn't mutate the instance of Subscription.
  • Update the CA certificate bundle.
  • Request sending logic unified across standard and custom methods (Unify request sending and result parsing logic #832)

@kamil-stripe
Copy link
Contributor

Copy link
Contributor

@dcr-stripe dcr-stripe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM overall!

Let's add a changelog entry for the request logic unification bit as well - a direct link to the PR is fine. This can help people track changes down in case something broke.

@stripe-ci stripe-ci assigned pakrym-stripe and unassigned dcr-stripe Aug 1, 2022
@pakrym-stripe pakrym-stripe merged commit fa26fa5 into master Aug 1, 2022
@@ -9,17 +9,16 @@ class UpdateableAPIResource(APIResource):
@classmethod
def modify(cls, sid, **params):
url = "%s/%s" % (cls.class_url(), quote_plus(util.utf8(sid)))
return cls._static_request("post", url, **params)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, what is the purpose of removing api_key support from the modify method?
How could I call modify without passing api_key explicitly?

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.

4 participants