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

Rename application_fee to application_fee_amount on PaymentIntent #552

Merged
merged 1 commit into from
Jul 12, 2018

Conversation

remi-stripe
Copy link
Contributor

We are renaming application_fee to application_fee_amount as a parameter and property. This is cleaner as application_fee usually is an Application Fee id and can be expanded.

We have confirmed no Java users have used this parameter yet so while a breaking change we can merge and release as the change is being deployed.

r? @brandur-stripe @ob-stripe
cc @stripe/api-libraries
cc @michelle-stripe @ln-stripe

@brandur-stripe
Copy link
Contributor

LGTM.

@brandur-stripe
Copy link
Contributor

We have confirmed no Java users have used this parameter yet so while a breaking change we can merge and release as the change is being deployed.

Just for clarity: are you okay with it if I just release this now? (It seems unlikely that this feature will pick up any new users between now and when the change is deployed to the API.)

@remi-stripe
Copy link
Contributor Author

@brandur-stripe yeah for once I would just merge early since the change is being deployed to prod soon! Thank you for the quick review.
Also, at worst the bug is that they can't see the app fee on the resource and since it's a beta, they would just ask why and we could explain quickly.

@brandur-stripe
Copy link
Contributor

Also, at worst the bug is that they can't see the app fee on the resource and since it's a beta, they would just ask why and we could explain quickly.

Yep, agreed. Okay, thanks!

@brandur-stripe brandur-stripe merged commit 4d22cc4 into master Jul 12, 2018
@brandur-stripe brandur-stripe deleted the fix-paymentintent-application-fee branch July 12, 2018 00:05
@brandur-stripe
Copy link
Contributor

Released as 5.50.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants