-
Notifications
You must be signed in to change notification settings - Fork 351
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
Stripe.Refund.list requires a charge #848
Comments
There is a workaround to define my own def list_refunds(params \\ %{}, opts \\ []) do
path = "/v1/refunds"
Stripe.Request.new_request(opts)
|> Stripe.Request.put_endpoint(path)
|> Stripe.Request.put_params(params)
|> Stripe.Request.put_method(:get)
|> Stripe.Request.make_request()
end So this is probably a low priority issue |
I've also stumbled upon this issue while trying to upgrade from stripity_stripe v"2.17.3" to "3.2.0". |
I have a lot of work to do at #843 Primarily, I need to readjust what I am doing and add another indirection that allows manual mapping of endpoints so that I can avoid breaking changes, |
This issue has been automatically marked as "stale:discard". We are sorry that we haven't been able to prioritize it yet. |
Closing this issue after a prolonged period of inactivity. If this issue is still relevant, feel free to re-open the issue. Thank you! |
Existing Issue or Pull Request Verification
Package Version
2.17.1
Are you using the latest version?
Steps to Reproduce
Invoke
Stripe.Refund.list()
and get error about function not existing.Try to invoke it with
Stripe.Refund.list(%{})
and get String replacement functionsExpected Result
Stripe's API supports listing all refunds without passing the charge parameter
Actual Result
The generated code requires charge to be passed in, possibly due to documentation bug on Stripe side?
Stripes docs show it's possible to request refunds without the charge ID:
The text was updated successfully, but these errors were encountered: