Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add support for the Issuing Dispute Submit API #676

Merged
merged 2 commits into from
Sep 2, 2020

Conversation

remi-stripe
Copy link
Contributor

r? @richardm-stripe
cc @stripe/api-libraries

"/v1/customers/cus_123/sources",
{"source": "tok_123"},
None,
"post", "/v1/charges/ch_123/refunds", {"amount": 123}, None,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"post", "/v1/charges/ch_123/refunds", {"amount": 123}, None,
"post",
"/v1/charges/ch_123/refunds",
{"amount": 123},
None,

Copy link
Contributor

@richardm-stripe richardm-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, CI is failing because it wants a formatting change I think

Copy link
Contributor

@cjavilla-stripe cjavilla-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

@remi-stripe
Copy link
Contributor Author

Nice catch richard, I always forget to run make fmt when I change the tests.

@remi-stripe remi-stripe merged commit 2d164cb into master Sep 2, 2020
@remi-stripe remi-stripe deleted the remi/codegen-474461f branch September 2, 2020 20:56
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants