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

Added requestCollection and requestSearchResult to StripeClientInterface #1832

Merged

Conversation

prathmesh-stripe
Copy link
Contributor

@prathmesh-stripe prathmesh-stripe commented Mar 27, 2025

Why?

We are adding new helper methods to StripeClientInterface to improve the return types in SDKs.
Stripe's implementation in BaseStripeClient.

See Also

https://jira.corp.stripe.com/browse/DEVSDK-2458#L80-L94

Changelog

  • ⚠️ Added requestSearchResult, requestCollection to StripeClientInterface. Developers building custom StripeClient will now have to implement these new methods.

@prathmesh-stripe prathmesh-stripe requested a review from a team as a code owner March 27, 2025 21:45
@prathmesh-stripe prathmesh-stripe requested review from xavdid-stripe and removed request for a team March 27, 2025 21:45
@prathmesh-stripe prathmesh-stripe enabled auto-merge (squash) March 27, 2025 21:51
@prathmesh-stripe prathmesh-stripe changed the base branch from master to latest-codegen-master March 27, 2025 22:04
auto-merge was automatically disabled March 27, 2025 22:26

Head branch was modified

@prathmesh-stripe prathmesh-stripe force-pushed the prathmesh/add-request-to-stripeclientinterface branch from 352fc1a to 7de4c7f Compare March 27, 2025 22:26
Copy link
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.

LGTM!

@prathmesh-stripe prathmesh-stripe changed the base branch from latest-codegen-master to master March 27, 2025 22:30
@prathmesh-stripe prathmesh-stripe enabled auto-merge (squash) March 27, 2025 22:30
@prathmesh-stripe prathmesh-stripe merged commit 81fb5c4 into master Mar 27, 2025
22 checks passed
@prathmesh-stripe prathmesh-stripe deleted the prathmesh/add-request-to-stripeclientinterface branch March 27, 2025 22:40
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.

2 participants