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

Codegen Stripe API #769

Closed
snewcomer opened this issue Dec 4, 2022 · 4 comments · Fixed by #776
Closed

Codegen Stripe API #769

snewcomer opened this issue Dec 4, 2022 · 4 comments · Fixed by #776

Comments

@snewcomer
Copy link
Collaborator

Putting an issue here to track work being done to codegen your APIs from the Stripe OpenAPI spec. This will automate version changes on the Stripe side and avoid issues with out of date library APIs and parameters.

Working to complete by end of month.

@maartenvanvliet
Copy link
Member

Hey. Wanted to let you know about the https://github.com/maartenvanvliet/striped library on I wrote. It generates a Stripe api Sdk from the spec at compile time, automatically incorporating changes from upstream.

It started out as a more generic approach to generating elixir openapi SDK's but I first wanted to focus on Stripe.
I saw this issue and wanted to keep you from doing duplicate work. Perhaps we can merge my library back into stripe-elixir.

See also https://hex.pm/packages/striped

@snewcomer
Copy link
Collaborator Author

snewcomer commented Dec 5, 2022

Thanks @maartenvanvliet! The work you did is really nice.

Ideally we have one elixir stripe library. As you probably know, an ecosystem with multiple libraries doing the same thing is messy for end users. I started generating some files with their spec today, but I'll certainly save myself the work if possible. Would love to merge your work into stripe-elixir if that is ok with you. How would you like to proceed?

(Starting to look at your library...)

@snewcomer
Copy link
Collaborator Author

snewcomer commented Dec 5, 2022

Just offering, happy to do the work to copy it in as well if that sat with you better as well depending on your current workload. Love what you did with that lib btw.

@maartenvanvliet
Copy link
Member

Fully agree with having one elixir stripe library, the community is best served when our collective efforts are focussed.
Feel free to copy it all in if you have the time.

There's no support for webhooks with plug like stripe-elixir currently has. So, that part can be kept.

We can work towards a v3-rc1 release of stripe-elixir. There's some work to be done still on file uploads and connected accounts but both shouldn't be a lot of work.

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

Successfully merging a pull request may close this issue.

2 participants