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

Introduce stripe.RawRequest as a canonical way to request APIs without definitions #1648

Merged
merged 24 commits into from
May 24, 2023

Conversation

richardm-stripe
Copy link
Contributor

@richardm-stripe richardm-stripe commented Apr 22, 2023

Adds a stripe.RawRequest method to be able to send untyped requests while still offering core library features like auth, retries, etc.

@anniel-stripe anniel-stripe force-pushed the richardm-raw-request branch from dbd7daf to d1ee60b Compare May 18, 2023 17:44
@anniel-stripe anniel-stripe changed the base branch from master to beta May 18, 2023 17:44
@anniel-stripe anniel-stripe marked this pull request as ready for review May 19, 2023 23:20
params.go Outdated Show resolved Hide resolved
preview/client.go Outdated Show resolved Hide resolved
params.go Outdated Show resolved Hide resolved
preview/client.go Outdated Show resolved Hide resolved
stripe.go Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
stripe.go Outdated Show resolved Hide resolved
@pakrym-stripe
Copy link
Contributor

Do we need developer_error handling as well?

@anniel-stripe
Copy link
Contributor

anniel-stripe commented May 24, 2023

Do we need developer_error handling as well?

I was thinking of doing that in a separate PR, but happy to put it in this one if you don't reviewing again.

@anniel-stripe anniel-stripe enabled auto-merge May 24, 2023 21:49
@anniel-stripe anniel-stripe disabled auto-merge May 24, 2023 21:49
@anniel-stripe anniel-stripe enabled auto-merge (squash) May 24, 2023 21:49
@anniel-stripe anniel-stripe merged commit 1de1537 into beta May 24, 2023
@anniel-stripe anniel-stripe deleted the richardm-raw-request branch May 24, 2023 21:49
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