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

Fixed phpdoc/constants for Customer, Invoice and TaxID #639

Merged
merged 1 commit into from
Apr 24, 2019
Merged

Fixed phpdoc/constants for Customer, Invoice and TaxID #639

merged 1 commit into from
Apr 24, 2019

Conversation

nickdnk
Copy link
Contributor

@nickdnk nickdnk commented Apr 24, 2019

In relation to customer tax ID logic, there have been a few changes to the customer object.

Updates reflected in this PR.

I removed the deprecated properties. Not sure if you want them to remain. I figure it's good practice to remove them as this draws attention to them having been deprecated without breaking anything.

Copy link
Contributor

@remi-stripe remi-stripe left a comment

Choose a reason for hiding this comment

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

Thanks for the fix as usual @nickdnk! Would you be up for also fixing Invoice since it got a lot of new properties too?

@nickdnk
Copy link
Contributor Author

nickdnk commented Apr 24, 2019

@remi-stripe

cracks knuckles - watch this!

@nickdnk
Copy link
Contributor Author

nickdnk commented Apr 24, 2019

Made a few adjustments. Namely to custom_fields that were annotated as mixed before - it is array - I missed that. And then I made a similar mistake in this PR on customer_address - it was a string, now mixed.

Should be good now.

Edit: I also adjusted TaxID phpdocs and added missing constants on verification.

@nickdnk nickdnk changed the title Fixed phpdoc for customer Fixed phpdoc/constants for Customer, Invoice and TaxID Apr 24, 2019
Fixed TaxID php docs and added constants
Fixed phpdocs for Invoice
Copy link
Contributor

@remi-stripe remi-stripe left a comment

Choose a reason for hiding this comment

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

Thanks for the quick work! I'll merge tomorrow with the TaxRate PR!

@remi-stripe remi-stripe merged commit a975640 into stripe:master Apr 24, 2019
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.

3 participants