Conversation
caseylocker
left a comment
There was a problem hiding this comment.
@matiasperrone-exo Quick fix:
Wrong Namespace in Security Schema
File: app/Swagger/Security/SummitSubmissionInvitationsAuthSchema.php
namespace App\Swagger\Security; // WRONG
Should be:
namespace App\Swagger\schemas; // CORRECT
caseylocker
left a comment
There was a problem hiding this comment.
Should have been a change request...
@matiasperrone-exo Quick fix:
Wrong Namespace in Security Schema
File: app/Swagger/Security/SummitSubmissionInvitationsAuthSchema.php
namespace App\Swagger\Security; // WRONG
Should be:
namespace App\Swagger\schemas; // CORRECT
|
Thanks @caseylocker for the comments. Now is ready to review again |
caseylocker
left a comment
There was a problem hiding this comment.
This has the same issue as a previous one in regards to OpenAPI 3.1 functionality being used in a 3.0 project in regards to the "tags" property.
SummitSpeakersSchemas.php:25 - Invalid OpenAPI 3.0 syntax:
items: new OA\Items(type: ['integer', 'string']),
Use oneOf format or change to a single type and document the expansion.
09fe403 to
6668141
Compare
caseylocker
left a comment
There was a problem hiding this comment.
With the new upgrade to OpenAPI 3.1, allowing type arrays, this should be good to go.
6668141 to
270bcec
Compare
Task:
Ref: https://app.clickup.com/t/86b6wkh7n