Skip to content

⚠️ Build SDK w/ V2 OpenAPI spec#3167

Merged
xavdid-stripe merged 2 commits intomasterfrom
DEVSDK-2750
Sep 9, 2025
Merged

⚠️ Build SDK w/ V2 OpenAPI spec#3167
xavdid-stripe merged 2 commits intomasterfrom
DEVSDK-2750

Conversation

@xavdid-stripe
Copy link
Copy Markdown
Contributor

@xavdid-stripe xavdid-stripe commented Sep 5, 2025

Why?

While SDK support for our v1 APIs comes from our public openapi spec, our v2 endpoints have been sources from an internal dump of the protobuf definitions of those APIs.

This PR marks our cutover point for our generation source: now we're using the (private, for now) V2 OpenAPI spec! There are a few changes/bugfixes as a result (documented below).

note: entire PR is codegen

What?

  • generated SDK using v2 openapi spec.

See Also

DEVSDK-2750

Changelog

  • ⚠️ The delete methods for v2 APIs (the ones in the StripeClient.v2 namespace) now return a V2DeletedObject which has the id of the object that has been deleted and a string representing the type of the object that has been deleted.

@xavdid-stripe xavdid-stripe marked this pull request as ready for review September 5, 2025 21:30
@xavdid-stripe xavdid-stripe requested a review from a team as a code owner September 5, 2025 21:30
@xavdid-stripe xavdid-stripe requested review from prathmesh-stripe and removed request for a team September 5, 2025 21:30
@xavdid-stripe xavdid-stripe enabled auto-merge (squash) September 5, 2025 21:30
@xavdid-stripe xavdid-stripe requested review from jar-stripe and removed request for prathmesh-stripe September 5, 2025 21:30
@xavdid-stripe xavdid-stripe changed the title Build SDK w/ V2 OpenAPI spec ⚠️ Build SDK w/ V2 OpenAPI spec Sep 5, 2025
Copy link
Copy Markdown
Contributor

@jar-stripe jar-stripe left a comment

Choose a reason for hiding this comment

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

Other than the DeletedObjects folder, it looks good!

Comment thread src/Stripe.net/Entities/V2/DeletedObjects/DeletedObject.cs
@xavdid-stripe xavdid-stripe merged commit 8ae0043 into master Sep 9, 2025
3 of 4 checks passed
@xavdid-stripe xavdid-stripe deleted the DEVSDK-2750 branch September 9, 2025 01:04
helenye-stripe pushed a commit that referenced this pull request Sep 23, 2025
* Add private-preview branch to CI workflow triggers

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* ⚠️ Build SDK w/ V2 OpenAPI spec (#3167)

* generate w/ v2 openapi spec

* move deleted object path

* Adds public BaseUrl to RawRequestOptions (#3170)

renames internal BaseUrl in RequestOptions to InternalBaseUrl 
adds public BaseUrl to RawRequestOptions - it delegates to InternalBaseUrl for safety

* Update generated code (#3172)

* Update generated code for v1955 and 2025-09-30.clover

* Update generated code for v1968 and 2025-09-30.clover

---------

Co-authored-by: Stripe OpenAPI <105521251+stripe-openapi[bot]@users.noreply.github.com>

* `just format` formats entire solution (#3174)

* Adds ability to specify file name and type when calling FileService.Create (#3171)

* Update generated code for 9de7288a5c444f47d15545549303c3de4c226c71 and 2025-09-30.clover

---------

Co-authored-by: Michael Broshi <mbroshi@stripe.com>
Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: David Brownman <109395161+xavdid-stripe@users.noreply.github.com>
Co-authored-by: jar-stripe <jar@stripe.com>
Co-authored-by: Stripe OpenAPI <105521251+stripe-openapi[bot]@users.noreply.github.com>
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.

3 participants