Skip to content

Feature | Extend Swagger Coverage for controller OAuth2SummitProposedScheduleAllowedLocationApiController#416

Merged
smarcet merged 3 commits intomainfrom
feature/openapi-documentation--oauth2summitproposedscheduleallowedlocationapicontroller
Oct 30, 2025
Merged

Feature | Extend Swagger Coverage for controller OAuth2SummitProposedScheduleAllowedLocationApiController#416
smarcet merged 3 commits intomainfrom
feature/openapi-documentation--oauth2summitproposedscheduleallowedlocationapicontroller

Conversation

@matiasperrone-exo
Copy link
Contributor

@matiasperrone-exo matiasperrone-exo self-assigned this Oct 15, 2025
@matiasperrone-exo matiasperrone-exo added the documentation Improvements or additions to documentation label Oct 15, 2025
@matiasperrone-exo matiasperrone-exo force-pushed the feature/openapi-documentation--oauth2summitproposedscheduleallowedlocationapicontroller branch from 3f86d7c to bc7b555 Compare October 17, 2025 20:09
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.

Since the existing tag naming pattern for the swagger documentation is Title Case with spaces please use that format for all tagging. See the existing "RSVP Invitations" tag as an example. In this case the format would be "Summit Proposed Schedule". The 'getter' => 'getTrack' correction is approved.

@matiasperrone-exo matiasperrone-exo force-pushed the feature/openapi-documentation--oauth2summitproposedscheduleallowedlocationapicontroller branch from bc7b555 to 758622e Compare October 29, 2025 20:28
@matiasperrone-exo
Copy link
Contributor Author

Since the existing tag naming pattern for the swagger documentation is Title Case with spaces please use that format for all tagging. See the existing "RSVP Invitations" tag as an example. In this case the format would be "Summit Proposed Schedule". The 'getter' => 'getTrack' correction is approved.

Thanks! @caseylocker this change is now implemented.

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.

I've reviewed the changes and tested locally. Approved for merge.

@smarcet smarcet merged commit bf9ee0a into main Oct 30, 2025
3 checks passed
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.

3 participants