Skip to content

Fix invalid example - #559

Closed
Maks3w wants to merge 1 commit into
OAI:masterfrom
Maks3w:patch-1
Closed

Fix invalid example#559
Maks3w wants to merge 1 commit into
OAI:masterfrom
Maks3w:patch-1

Conversation

@Maks3w

Copy link
Copy Markdown
Contributor

http://online.swagger.io/validator?url=https://raw.githubusercontent.com/OAI/OpenAPI-Specification/master/examples/v2.0/json/petstore-with-external-docs.json

Swagger Error
Missing required property definition: name
Jump to line 147
Details
Object
code: "OBJECT_MISSING_REQUIRED_PROPERTY_DEFINITION"
message: "Missing required property definition: name"
path: Array [4]
level: 900
type: "Swagger Error"
description: "Missing required property definition: name"
lineNumber: 147

http://online.swagger.io/validator?url=https://raw.githubusercontent.com/OAI/OpenAPI-Specification/master/examples/v2.0/json/petstore-with-external-docs.json
> Swagger Error
Missing required property definition: name
Jump to line 147
Details
Object
code: "OBJECT_MISSING_REQUIRED_PROPERTY_DEFINITION"
message: "Missing required property definition: name"
path: Array [4]
level: 900
type: "Swagger Error"
description: "Missing required property definition: name"
lineNumber: 147
@jharmn

Copy link
Copy Markdown
Contributor

FYI this and many other issues in samples are awaiting review/merge in https://github.com/OAI/OpenAPI-Specification/pull/389/files
/cc @webron

@Maks3w

Copy link
Copy Markdown
ContributorAuthor

Thx. I'll keep this open because its more simple and has a better chance of be merged.

Could you edit #389 description and append to the end Close #559? So if your PR is merged before this is closed

@jharmn

Copy link
Copy Markdown
Contributor

@Maks3w done. TIL how to close related PRs with commit messages ;)

@Maks3w
Maks3w deleted the patch-1 branch February 13, 2016 07:45
AndersDJohnson pushed a commit to AndersDJohnson/OpenAPI-Specification that referenced this pull request Apr 8, 2019
Cleaning up samples, parity between yaml and json - CloseOAI#559
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@Maks3w@jharmn