Skip to content

Feature | Add openapi documentation for OAuth2SummitSubmittersApiController#457

Merged
smarcet merged 2 commits intomainfrom
feature/add-summit-submitter-swagger
Dec 8, 2025
Merged

Feature | Add openapi documentation for OAuth2SummitSubmittersApiController#457
smarcet merged 2 commits intomainfrom
feature/add-summit-submitter-swagger

Conversation

@andrestejerina97
Copy link
Contributor

@andrestejerina97 andrestejerina97 marked this pull request as ready for review November 13, 2025 20:05
@andrestejerina97 andrestejerina97 force-pushed the feature/add-summit-submitter-swagger branch from 6c90063 to 49a3bef Compare November 13, 2025 20:08
@andrestejerina97 andrestejerina97 self-assigned this Nov 13, 2025
@andrestejerina97 andrestejerina97 added the documentation Improvements or additions to documentation label 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.

Quick formatting update for title case and then it should be good to go.
Current: tags: ["SummitSubmitters"]
Expected: tags: ["Summit Submitters"]

Also add operationId where needed.

@matiasperrone-exo
Copy link
Contributor

@caseylocker the requested change is done.

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

Copy link
Collaborator

@smarcet smarcet left a comment

Choose a reason for hiding this comment

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

LGTM

@smarcet smarcet merged commit abe66f4 into main Dec 8, 2025
4 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.

5 participants