-
Notifications
You must be signed in to change notification settings - Fork 725
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
feat: enable Rest api using grpc-gateway #834
Conversation
Signed-off-by: Sahil Badla <sahil_badla@intuit.com>
Signed-off-by: Sahil Badla <sahil_badla@intuit.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the possibility to have a rest/http-api, made some suggestions for naming and I think you need to rebase
Thank you for this contribution!
Signed-off-by: Sahil Badla <sahil_badla@intuit.com>
Signed-off-by: Sahil Badla <sahil_badla@intuit.com>
Signed-off-by: Sahil Badla <sahil_badla@intuit.com>
Signed-off-by: Sahil Badla <sahil_badla@intuit.com>
Signed-off-by: Sahil Badla <146279034+sbadla1@users.noreply.github.com>
@AlexsJones @thschue Thanks for the review. Please note that my changes need to reference methods(example) generated by grpc-gateway. Grpc-gateway changes and code generation part is in this PR |
Sorry for this @sbadla1 going to get this sorted out today, updating our schemas |
Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
I have updated and resolved issues with the code but |
Signed-off-by: Alex Jones <alexsimonjones@gmail.com>
Signed-off-by: Sahil Badla <sahil_badla@intuit.com>
Signed-off-by: Sahil Badla <sahil_badla@intuit.com>
Signed-off-by: Sahil Badla <sahil_badla@intuit.com>
Signed-off-by: Sahil Badla <sahil_badla@intuit.com>
Signed-off-by: Sahil Badla <sahil_badla@intuit.com>
Signed-off-by: Sahil Badla <sahil_badla@intuit.com>
Closes #
π Description
enable Rest api using grpc-gateway. This is related to PR k8sgpt-ai/schemas#20
β Checks
βΉ Additional Information