Skip to content

fix: clarify support for Boolean JSON Schemas#561

Merged
magicmatatjahu merged 1 commit into
asyncapi:2021-06-releasefrom
magicmatatjahu:clarify-boolean-schemas
Jun 15, 2021
Merged

fix: clarify support for Boolean JSON Schemas#561
magicmatatjahu merged 1 commit into
asyncapi:2021-06-releasefrom
magicmatatjahu:clarify-boolean-schemas

Conversation

@magicmatatjahu

Copy link
Copy Markdown
Member

title: "Mention about true/false JSON Schemas in Schema Object"

As Schema Object in spec is defined as superset of the JSON Schema Specification Draft 07, it would be nice to specify that the Schema Object itself can be boolean value. I know it results from the definition of draft 07, so I'd like to hear your opinion about the changes.

Description for true/false schemas I took from https://github.com/OAI/OpenAPI-Specification/pull/2609/files Thanks to @MikeRalphson

NOTE: PR is a copy of #550 but with changed base branch from master to 2021-06-release - remote git see other revisions than I have in local and tries to interpolate the commits from master branch


Related issue(s):
asyncapi/parser-js#232
Fixes #554

Related PR(s):
asyncapi/spec-json-schemas#63 - fix bug with true/false schemas in JSON Schema of spec
asyncapi/parser-js#311 - handle true/false JSON Schema in parser
asyncapi/asyncapi-react#367 - handle Boolean JSON Schema in the react-component


@magicmatatjahu magicmatatjahu added the ✏️ Editorial PR is non-normative or does not influence implementation label Jun 15, 2021
@sonarqubecloud

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@magicmatatjahu magicmatatjahu requested a review from derberg June 15, 2021 12:09
@magicmatatjahu magicmatatjahu merged commit f30dccf into asyncapi:2021-06-release Jun 15, 2021
@magicmatatjahu magicmatatjahu deleted the clarify-boolean-schemas branch June 15, 2021 12:20
@asyncapi-bot

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.1.0-2021-06-release.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@asyncapi-bot

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.2.0-2021-06-release.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✏️ Editorial PR is non-normative or does not influence implementation released on @2021-06-release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants