Closed MikeRalphson closed 7 years ago
Hi,
I had started converting the petstore spec to openapi 3.0 but had not finished. 30 min before you posted this issue, I discovered your project and used it to convert the petstore.json file to openapi 3.0 spec :).
The next version of the petstore.json file in the test folder will be this version converted with your tool.
The converted file now highlight cases that we do not handle, I will work on this and commit updates in few hours.
That's very cool. Please do let me know if anything in the converted sample doesn't look right to you.
Just to make you aware, spotted a (big) bug in the converter. parameter.type
has been removed, all parameters now need a schema
property if they are typed. Will let you know when examples updated.
Ok, saw the problem in path /store/order/{orderId}.
There is something I'm missing in the spec. Schema seems not mandatory, but parameters should have a default type (string ?).
I didn't see your bug because I added a default schema in preprocessing. I will keep this default schema with a string type until this get more precise in the spec.
I agree, I think the spec should explicitly define untyped parameters as accepting a string with no defined format restrictions. Defining a schema seems the only way to set a default value etc.
Converter/validator passing test suite again, as of v1.2.0.
Updated the example file using the last version of your converter.
I've been following your project with keen interest, as I am writing a converter (and validator) for Swagger 2.0 to OpenAPI 3.0.x.
I'm not sure if this is the right time to raise this issue, as you may not want to track Release Candidate versions of the specification, and even if so, the error may be in my validator or my reading of the OpenAPI Specification.
There is a hosted version of the validator tool here which has an API for validating OpenAPI 3.0.x specs.
The result of running against your spec is as reported here (first error only):
Thus the badge functionality currently returns
I have tried comparing your v3 Petstore spec to my converted version, and other issues include
formData
parameters, which have been removed from the spec.Keen to hear your thoughts.