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

Swagger Auth Header Code Generation Updates #33

Merged
merged 6 commits into from
Oct 21, 2022

Conversation

atulsriv
Copy link
Contributor

No description provided.

@atulsriv atulsriv marked this pull request as ready for review October 19, 2022 14:07
@atulsriv atulsriv changed the title Trad 206/swagger auth header Swagger Auth Header Code Generation Updates Oct 19, 2022
@atulsriv atulsriv requested review from aspin and yobloxroute October 19, 2022 14:21
proto/api.proto Outdated
@@ -36,6 +55,11 @@ service Api {
get: "/api/v1/market/price"
};
option (grpc.gateway.protoc_gen_openapiv2.options.openapiv2_operation) = {
security: {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

think these lines below are unnecessary

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, been updated

@atulsriv atulsriv force-pushed the TRAD-206/swagger-auth-header branch from cc770aa to 4fe91b6 Compare October 21, 2022 18:16
@atulsriv atulsriv merged commit 3118aee into develop Oct 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants