Skip to content

Feature | Extend Swagger Coverage for controller OAuth2SummitRegistrationCompaniesApiController - #459

Merged
romanetar merged 6 commits into
mainfrom
feature/add-registration-companies-swagger
Jan 28, 2026
Merged

Feature | Extend Swagger Coverage for controller OAuth2SummitRegistrationCompaniesApiController#459
romanetar merged 6 commits into
mainfrom
feature/add-registration-companies-swagger

Conversation

@andrestejerina97

Copy link
Copy Markdown
Contributor

@andrestejerina97
andrestejerina97 force-pushed the feature/add-registration-companies-swagger branch from e6d4149 to a360faa Compare November 14, 2025 18:47
@andrestejerina97
andrestejerina97 marked this pull request as ready for review November 14, 2025 18:48

@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.

Change tags to
tags: ["Registration Companies"], // Add space

Should be good after that.

@matiasperrone-exo matiasperrone-exo changed the title Feature | Add openapi documentation for OAuth2SummitRegistrationCompaniesApiController Feature | Extend Swagger Coverage for controller OAuth2SummitRegistrationCompaniesApiController Dec 9, 2025
@matiasperrone-exo matiasperrone-exo added the documentation Improvements or additions to documentation label Dec 9, 2025
@matiasperrone-exo
matiasperrone-exo force-pushed the feature/add-registration-companies-swagger branch from a360faa to 461c11d Compare December 9, 2025 21:02
@smarcet
smarcet force-pushed the main branch 4 times, most recently from c6ecdd0 to 728ae67 Compare December 17, 2025 00:43

@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.

Build failure. Swagger generation fails with duplicate class declaration

Cannot declare class App\Swagger\schemas\CompanySchema, because the name is already in use

@matiasperrone-exo
matiasperrone-exo force-pushed the feature/add-registration-companies-swagger branch 2 times, most recently from adfc50a to ef3ff95 Compare December 29, 2025 22:43
@matiasperrone-exo

Copy link
Copy Markdown
Contributor

@smarcet it is fixed now

@matiasperrone-exo
matiasperrone-exo force-pushed the feature/add-registration-companies-swagger branch 2 times, most recently from acd6fba to cdf0027 Compare January 28, 2026 12:31
andrestejerina97 and others added 6 commits January 28, 2026 15:37
Signed-off-by: Matias Perrone <github@matiasperrone.com>
Signed-off-by: Matias Perrone <github@matiasperrone.com>
Signed-off-by: Matias Perrone <github@matiasperrone.com>
@matiasperrone-exo
matiasperrone-exo force-pushed the feature/add-registration-companies-swagger branch from cdf0027 to 3f6b87f Compare January 28, 2026 15:38

@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.

LGTM

@romanetar
romanetar merged commit 5fed732 into main Jan 28, 2026
5 checks passed
@romanetar
romanetar deleted the feature/add-registration-companies-swagger branch January 28, 2026 17:59
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