Feature | Extend Swagger Coverage for controller OAuth2SummitsTicketTypesApiController

[matiasperrone-exo]

Task:

Ref: https://app.clickup.com/t/86b6wkhah

[caseylocker]

@matiasperrone-exo
Wrong namespace in security schema:
// Current (line 3)
namespace App\Swagger\Security;

// Should be
namespace App\Swagger\schemas;

Wrong x: parameter key (6 occurrences): Lines 297, 558, 620, 689, 738, 799 all have:
// Current
x: ["authz_groups" => [IGroup::SuperAdmins, IGroup::Administrators, IGroup::SummitAdministrators, IGroup::SummitRegistrationAdmins]],

// Should be
x: ["required-groups" => [IGroup::SuperAdmins, IGroup::Administrators, IGroup::SummitAdministrators, IGroup::SummitRegistrationAdmins]],

[matiasperrone-exo]

Thanks @caseylocker for the comments. Now is ready to review again

[caseylocker]

@matiasperrone-exo regenerate the docs locally and you'll see the issue.

In SummitRegistrationSchemas.php line 98:

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

[caseylocker]

approved

[caseylocker]

@matiasperrone-exo
Generation is failing from a duplicate operationID. Please see test output.

operationId must be unique. Duplicate value found: "getAllTicketTypesBySummit"

[matiasperrone-exo]

Thanks @caseylocker for the comments. Now is ready to review again