Skip to content

Oas31 schema - #1

Merged
philsturgeon merged 8 commits into
philsturgeon:oas31-schemafrom
jdesrosiers:oas31-schema
Feb 23, 2021
Merged

Oas31 schema#1
philsturgeon merged 8 commits into
philsturgeon:oas31-schemafrom
jdesrosiers:oas31-schema

Conversation

@jdesrosiers

@jdesrosiersjdesrosiers commented Feb 19, 2021

Copy link
Copy Markdown

That fixes all the issues with references. I haven't tried running it against an actual OpenAPI document yet.

#ExternalDocumentation is referenced, but never defined. I copied the schema from the 3.0 meta-schema

@jdesrosiers
jdesrosiersforce-pushed the oas31-schema branch 2 times, most recently from 2212d51 to 796f71fCompareFebruary 19, 2021 17:46
@jdesrosiers
jdesrosiersforce-pushed the oas31-schema branch 2 times, most recently from 855dfbd to 8c70601CompareFebruary 19, 2021 19:25
@jdesrosiers

Copy link
Copy Markdown
Author

Added a couple of fixes based on @MikeRalphson's openapi3.1-examples. There's one that still fails, but it's something that JSON Schema can't express.

Apparently there is no schema for webhooks yet. I put a placeholder in with a TODO comment.

@jdesrosiers

Copy link
Copy Markdown
Author

Once OAI#2467 is merged, we can update /$defs/Objects/$defs/Schema to point to the dialect.

@philsturgeon
philsturgeon merged commit c102f7d into philsturgeon:oas31-schemaFeb 23, 2021
@philsturgeon

Copy link
Copy Markdown
Owner

Thank you @jdesrosiers, I'll keep plugging away at this when I can and I'll keep an eye on OAI#2467

@jdesrosiers
jdesrosiers deleted the oas31-schema branch September 20, 2021 18:27
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

@jdesrosiers@philsturgeon