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

Product and plan support #518

Merged

Conversation

pantera-stripe
Copy link
Contributor

@pantera-stripe pantera-stripe commented Feb 14, 2018

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:

  • Allows plans to be created with products (with a product ID)
  • Allows plans to be created with inline product fields

r? @brandur-stripe @tmaxwell-stripe
cc @stripe/api-libraries

@pantera-stripe pantera-stripe changed the title WIP Product and plan support Product and plan support Feb 14, 2018
@@ -23,7 +23,7 @@ cache:

env:
global:
- STRIPE_MOCK_VERSION=0.4.0
- STRIPE_MOCK_VERSION=0.7.0
Copy link
Contributor Author

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",
Copy link
Contributor Author

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.

@pantera-stripe
Copy link
Contributor Author

Also...
r? @ob-stripe

@ob-stripe
Copy link
Contributor

LGTM!

@ob-stripe ob-stripe merged commit 8dbdb3e into stripe:master Feb 15, 2018
nadaismail-stripe pushed a commit that referenced this pull request Oct 18, 2024
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] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

6 participants