Skip to content

Conversation

@jdesrosiers
Copy link

@jdesrosiers jdesrosiers commented Feb 19, 2021

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 jdesrosiers force-pushed the oas31-schema branch 2 times, most recently from 2212d51 to 796f71f Compare February 19, 2021 17:46
@jdesrosiers jdesrosiers force-pushed the oas31-schema branch 2 times, most recently from 855dfbd to 8c70601 Compare February 19, 2021 19:25
@jdesrosiers
Copy link
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
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-schema Feb 23, 2021
@philsturgeon
Copy link
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 free to 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