Skip to content

Conversation

@andrestejerina97
Copy link
Contributor

@andrestejerina97 andrestejerina97 marked this pull request as ready for review November 12, 2025 20:08
@andrestejerina97 andrestejerina97 force-pushed the feature/add-openapi-documentation-paymentgatewaywebhook-controller branch from e5f9051 to b865897 Compare November 13, 2025 17:53
@andrestejerina97 andrestejerina97 changed the title feat: Extend Swagger Coverage for controller PaymentGatewayWebHookController Feature | Extend Swagger Coverage for controller PaymentGatewayWebHookController Nov 13, 2025
Copy link

@caseylocker caseylocker left a 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.

Copy link

@caseylocker caseylocker left a 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.

@matiasperrone-exo matiasperrone-exo force-pushed the feature/add-openapi-documentation-paymentgatewaywebhook-controller branch from b865897 to a262fad Compare December 8, 2025 14:40
…controller' of github-tipit:OpenStackweb/summit-api into feature/add-openapi-documentation-paymentgatewaywebhook-controller
@matiasperrone-exo
Copy link
Contributor

@caseylocker seems that there were two branches for the same controller, now those branches were merged. Now you should see the code as requested.

Copy link

@caseylocker caseylocker left a comment

Choose a reason for hiding this comment

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

Approved

@matiasperrone-exo matiasperrone-exo added the documentation Improvements or additions to documentation label Dec 9, 2025
@smarcet smarcet force-pushed the main branch 4 times, most recently from c6ecdd0 to 728ae67 Compare December 17, 2025 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants