Skip to content

Conversation

@andrestejerina97
Copy link
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
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.

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

Build failure. Swagger generation fails with duplicate class declaration

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

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