-
Notifications
You must be signed in to change notification settings - Fork 2
Feature | Extend Swagger Coverage for controller PaymentGatewayWebHookController #455
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
base: main
Are you sure you want to change the base?
Feature | Extend Swagger Coverage for controller PaymentGatewayWebHookController #455
Conversation
e5f9051 to
b865897
Compare
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.
This is very close but for consistency we need to update the tag to adhere to our current Title Case format.
Current:
tags: ["PaymentGatewayHook"]
Update to:
tags: ["Payment Gateway Webhook"]
Also add operationId where needed.
Once that's done it should be good to go.
…rformance chore: add needed IDX
caseylocker
left a comment
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.
This is very close but for consistency we need to update the tag to adhere to our current Title Case format.
Current:
tags: ["PaymentGatewayHook"]
Update to:
tags: ["Payment Gateway Webhook"]
Also add operationId where needed.
Once that's done it should be good to go.
b865897 to
a262fad
Compare
…controller' of github-tipit:OpenStackweb/summit-api into feature/add-openapi-documentation-paymentgatewaywebhook-controller
|
@caseylocker seems that there were two branches for the same controller, now those branches were merged. Now you should see the code as requested. |
caseylocker
left a comment
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.
Approved
c6ecdd0 to
728ae67
Compare
ref: https://app.clickup.com/t/86b6r284x