-
Notifications
You must be signed in to change notification settings - Fork 76
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
Swagger to notebook to microservice #2
Comments
/cc @Lull3rSkat3r |
Discussed on appear. We're going to axe the user permissions and auth from the pet store example since it starts to conflate microservice with web app backend. We're going to put in the PR without the error codes and headers to get a MVP in. We'll keep this open to keep improving it over time. |
[Issue #2]: Created swagger codegen module to generate notebooks.
Tried the instructions today. Worked fine. Good work. Other than the API completion that we talked about, what's missing is a separate UI so folks can understand how the microservice can be used. I really don't want to implement a whole pet store web site, but we have to have something that demonstrates how notebook-to-microservice = backend that can then be used to develop a frontend (be it by coding one or using some dashboarding tool). I have no thoughts at the moment other than "write something simple ourselves". Any ideas? |
So a few ideas (in order of what I think is best):
|
#1 sounds fine for this demo. How about just bringing it up in a second container along side the first microservice one and telling people to point their browsers at it? |
Brining up another container would result in the CORS issues I was seeing in the second point. |
We decided to enable the CORS headers in kernel gateway over notebook-http mode. That way we can use the public editor.swagger.io. |
We enabled the CORS headers in the kernel gateway for notebook-http mode. This now works as expected on editor.swagger.io. Closing. |
Demonstrate turning a Swagger spec into kernel-based microservice. Include:
The text was updated successfully, but these errors were encountered: