-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Default Swagger support in server/v2? #23020
Comments
If we agree on not adding it, I can mention this in the v2 docs: #22531 However, we could easily create a server/v2/api/swagger if that's something we still want to support out of the box. My two cents would say we do not need that, and that less is more in that case. |
It matters how grpc-gateway will work, right now its generated from there tooling but if we remove the tooling then would it work the same way with generation? |
Taking over #23092 and finishing it. |
Noticed while doing #23018 that server/v2 doesn't server a swagger UI.
Is that something server/v2 should support out of the box?
The server in v1 is fairly simple: https://github.com/cosmos/cosmos-sdk/blob/d297747/server/swagger.go, hence super easily to add by chain themselves with the UI they prefer.
The text was updated successfully, but these errors were encountered: