-
Notifications
You must be signed in to change notification settings - Fork 460
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
Product and plan support #518
Product and plan support #518
Conversation
9bd347c
to
c254932
Compare
c254932
to
9bb1c27
Compare
@@ -23,7 +23,7 @@ cache: | |||
|
|||
env: | |||
global: | |||
- STRIPE_MOCK_VERSION=0.4.0 | |||
- STRIPE_MOCK_VERSION=0.7.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This and the version bump are the largest changes from the original PR.
@@ -17,7 +17,7 @@ func TestEphemeralKeyDel(t *testing.T) { | |||
func TestEphemeralKeyNew(t *testing.T) { | |||
key, err := New(&stripe.EphemeralKeyParams{ | |||
Customer: "cus_123", | |||
StripeVersion: "2017-05-25", | |||
StripeVersion: "2018-02-06", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Figured I should bump this to the latest API version. Plan and Product updates reflect these changes with the removal of name
and statement_descriptor
from Plan
.
Also... |
LGTM! |
Bumps [sorbet](https://github.com/sorbet/sorbet) from 0.5.10096 to 0.5.10109. - [Release notes](https://github.com/sorbet/sorbet/releases) - [Commits](https://github.com/sorbet/sorbet/commits) --- updated-dependencies: - dependency-name: sorbet dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
All credit goes to @jhoon-stripe for the original PR: #496
I just rebased and resolved conflicts. From the original PR:
Summary
This PR implements upcoming changes to the way in which plan objects are created in the Stripe API. Specifically, it:
r? @brandur-stripe @tmaxwell-stripe
cc @stripe/api-libraries