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 CreditNoteLineItem #1013

Merged
merged 1 commit into from
Jan 14, 2020
Merged

Conversation

remi-stripe
Copy link
Contributor

@remi-stripe remi-stripe commented Jan 12, 2020

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

@remi-stripe remi-stripe force-pushed the remi-wip-credit-note-line branch from 271fa5f to 9de0209 Compare January 14, 2020 02:13
@remi-stripe remi-stripe changed the title WIP creditnote line item Add support for CreditNoteLineItem Jan 14, 2020
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.

Added a few of my naive questions :)

creditnote.go Show resolved Hide resolved
TaxRates []*TaxRate `json:"tax_rates"`
Type CreditNoteLineItemType `json:"type"`
UnitAmount int64 `json:"unit_amount"`
UnitAmountDecimal float64 `json:"unit_amount_decimal,string"`
Copy link
Contributor

Choose a reason for hiding this comment

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

not familiar with this string thing at the end so pointing it out. guessing that's how we specify the type of the json field and that'll be converted to float64.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, that's mostly cargo culted from other resources

creditnote.go Show resolved Hide resolved
creditnote.go Show resolved Hide resolved
creditnote.go Outdated
Amount *int64 `form:"amount"`
CreditAmount *int64 `form:"credit_amount"`
Invoice *string `form:"invoice"`
Memo *string `form:"memo"`
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this one have Lines too?

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.

Cool. lgtm

@remi-stripe remi-stripe merged commit 119b386 into master Jan 14, 2020
@remi-stripe remi-stripe deleted the remi-wip-credit-note-line branch January 14, 2020 22:45
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.

4 participants