Feature | Extend Swagger Coverage for controller OAuth2SummitMediaFileTypeApiController

[matiasperrone-exo]

Task:

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

[caseylocker]

@matiasperrone-exo looks like this one also relies on 'summit_rsvp_oauth2' being a defined security schema but it doesn't exist in the Pr in app/Swagger/schemas.php
It looks like once that's actually defined it will fix at least a few of these PRs.

[matiasperrone-exo]

@matiasperrone-exo looks like this one also relies on 'summit_rsvp_oauth2' being a defined security schema but it doesn't exist in the Pr in app/Swagger/schemas.php It looks like once that's actually defined it will fix at least a few of these PRs.

@caseylocker thanks the issue is now fixed and the docs are compiling now.

[matiasperrone-exo]

@caseylocker @smarcet All the scopes are now the right ones, please review.

[caseylocker]

Change the "tags" for the GET all to match the other ones - tags: ['Summit Media File Types'],

None of the 5 endpoints have operationId defined. Add for API client generation:
GET all: operationId: 'getAllSummitMediaFileTypes'
GET one: operationId: 'getSummitMediaFileType'
POST: operationId: 'createSummitMediaFileType'
PUT: operationId: 'updateSummitMediaFileType'
DELETE: operationId: 'deleteSummitMediaFileType'

Missing type: 'object' in Paginated Schema
Should be:
new OA\Schema(
type: 'object',
properties: [

[matiasperrone-exo]

Change the "tags" for the GET all to match the other ones - tags: ['Summit Media File Types'],

None of the 5 endpoints have operationId defined. Add for API client generation: GET all: operationId: 'getAllSummitMediaFileTypes' GET one: operationId: 'getSummitMediaFileType' POST: operationId: 'createSummitMediaFileType' PUT: operationId: 'updateSummitMediaFileType' DELETE: operationId: 'deleteSummitMediaFileType'

Missing type: 'object' in Paginated Schema Should be: new OA\Schema( type: 'object', properties: [

@caseylocker this PR is ready to be reviewed, I've included all the requested changes

[caseylocker]

@matiasperrone-exo Right on. Looks good. @smarcet if you agree please merge.

[smarcet]

LGTM