search

jagql / framework

A config driven NodeJS framework implementing json:api and GraphQL
MIT License
55 stars 20 forks source link

Semantic error with array attributes #54

Open gboston opened 6 years ago

gboston commented 6 years ago

When copying the swagger.json from the example, I get the following semantic error:

Semantic error at definitions.photos.properties.attributes.properties.tags Schemas with 'type: array', require a sibling 'items: ' field

Seems like the tags attribute doesn't get the required items field.

championswimmer commented 6 years ago

Yeah, we are considering moving to Swagger v3 Among other things, this error will also get solved

On Wed 16 May, 2018, 1:22 AM Glenn Bostoen, notifications@github.com wrote:

When copying the swagger.json from the example, I get the following semantic error:

Semantic error at definitions.photos.properties.attributes.properties.tags Schemas with 'type: array', require a sibling 'items: ' field

Seems like the tags attribute doesn't get the required items field.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jagql/framework/issues/54, or mute the thread https://github.com/notifications/unsubscribe-auth/ABQ_ygUKobdUhiRYDEZdUUvVItDvA9cxks5tyzIYgaJpZM4UAK1N .

gboston commented 6 years ago

We're using Apigee and this doesn't support Swagger v3. I think this applies to a lot of tooling that doesn't support Swagger v3. Will future updates still support Swagger v2?

championswimmer commented 6 years ago

We will support both, don't worry :)