Skip to content

Feature | Add openapi documentation for OAuth2SummitSubmittersApiController - #457

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

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

Conversation

@andrestejerina97

Copy link
Copy Markdown
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

@caseylocker caseylocker left a comment

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor

@caseylocker the requested change is done.

@caseylocker caseylocker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved

@smarcet smarcet left a comment

Copy link
Copy Markdown
Collaborator

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