Add support for OpenAPI v3. #59

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

soyuka merged 4 commits into api-platform:master from bpolaszek:openapi3

Sep 17, 2019

Merged

Add support for OpenAPI v3. #59

soyuka merged 4 commits into api-platform:master from bpolaszek:openapi3

Sep 17, 2019

Conversation

@bpolaszek

Copy link

Contributor

@bpolaszek bpolaszek commented Sep 3, 2019

This PR adds a parser for OpenAPI v3.

@dunglas Can you please review?

@bcobzh


 Add support for OpenAPI v3.

b570232

@bpolaszek

Copy link

Contributor Author

bpolaszek commented Sep 3, 2019

Fix small typo in README.

dunglas

dunglas requested changes

Sep 3, 2019

View reviewed changes

README.md

parseSwaggerDocumentation('https://demo.api-platform.com/docs.json').then(({api}) => console.log(api));

```

**OpenApi v3**

Copy link

Member

@dunglas dunglas Sep 3, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can promote it over Swagger

Copy link

Contributor Author

@bpolaszek bpolaszek Sep 4, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you mean we should remove everything related to Swagger in the Readme?

README.md Outdated

[![MIT Licence](https://badges.frapsoft.com/os/mit/mit.svg?v=103)](https://opensource.org/licenses/mit-license.php)

`api-doc-parser` is a JavaScript (ES6) library to parse [Hydra](http://hydra-cg.com) or Swagger API documentations and transform them

`api-doc-parser` is a JavaScript (ES6) library to parse [Hydra](http://hydra-cg.com), Swagger and OpenAPI documentations and transform them

Copy link

Member

@dunglas dunglas Sep 3, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a link to OpenAPI?

Copy link

Contributor Author

@bpolaszek bpolaszek Sep 4, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure.

package.json Outdated

{

"name": "@api-platform/api-doc-parser",

"version": "0.7.2",

"version": "0.7.3",

Copy link

Member

@dunglas dunglas Sep 3, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you revert this? It's the CI that takes care of this.

Copy link

Contributor Author

@bpolaszek bpolaszek Sep 4, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great 👍

@bpolaszek bpolaszek mentioned this pull request

Sep 4, 2019

Add OpenAPI v3 support. api-platform/create-client#146

Merged

bcobzh and others added 3 commits

September 6, 2019 14:13

@bcobzh @bpolaszek


 Add support for OpenAPI v3.

daadd41

@bpolaszek


 Use lodash to ensure required fields are actually an array

fe5cacf

@bpolaszek


 Fix package.json

@soyuka soyuka merged commit d13b41e into api-platform:master

Sep 17, 2019

@soyuka

Copy link

Member

soyuka commented Sep 17, 2019

thanks @bpolaszek !

@alanpoulain alanpoulain mentioned this pull request

May 13, 2020

Add OpenAPI v3 support #48

Closed

Labels

None yet

4 participants

@bpolaszek @soyuka @dunglas @bcobzh

Uh oh!

Add support for OpenAPI v3. #59

Add support for OpenAPI v3. #59

Uh oh!

Conversation

@bpolaszek bpolaszek commented Sep 3, 2019

Uh oh!

bpolaszek commented Sep 3, 2019

Uh oh!

@dunglas dunglas Sep 3, 2019

Choose a reason for hiding this comment

Uh oh!

@bpolaszek bpolaszek Sep 4, 2019

Choose a reason for hiding this comment

Uh oh!

@dunglas dunglas Sep 3, 2019

Choose a reason for hiding this comment

Uh oh!

@bpolaszek bpolaszek Sep 4, 2019

Choose a reason for hiding this comment

Uh oh!

@dunglas dunglas Sep 3, 2019

Choose a reason for hiding this comment

Uh oh!

@bpolaszek bpolaszek Sep 4, 2019

Choose a reason for hiding this comment

Uh oh!

soyuka commented Sep 17, 2019

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants