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

Add support for CreditNote #1564

Merged
merged 3 commits into from
Apr 18, 2019
Merged

Add support for CreditNote #1564

merged 3 commits into from
Apr 18, 2019

Conversation

remi-stripe
Copy link
Contributor

cc @stripe/api-libraries

@remi-stripe remi-stripe force-pushed the remi-add-credit-notes branch 2 times, most recently from 3ee4f6d to a882abc Compare March 30, 2019 01:46
@remi-stripe
Copy link
Contributor Author

r? @ob-stripe

src/Stripe.net/Entities/CreditNote.cs Outdated Show resolved Hide resolved
src/Stripe.net/Entities/CreditNote.cs Outdated Show resolved Hide resolved
@ob-stripe ob-stripe assigned remi-stripe and unassigned ob-stripe Apr 17, 2019
@remi-stripe remi-stripe changed the title [WIP] Add support for CreditNote Add support for CreditNote Apr 17, 2019
@remi-stripe
Copy link
Contributor Author

@ob-stripe Thanks for the review. I made some changes (and added 2 missing fields). I did not force push to make the re-review easier (and I'll squash after approval).

Can you re-r? the last 2 commits?

@remi-stripe remi-stripe assigned ob-stripe and unassigned remi-stripe Apr 17, 2019
Copy link
Contributor

@ob-stripe ob-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, though the new invoice fields are not in the public spec yet.

@remi-stripe
Copy link
Contributor Author

LGTM, though the new invoice fields are not in the public spec yet.

Yep those are tricky to align unfortunately, I originally missed them

@remi-stripe remi-stripe merged commit 158ee1b into master Apr 18, 2019
@remi-stripe remi-stripe deleted the remi-add-credit-notes branch April 18, 2019 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants