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
Fix validation errors for Swagger Open API #11228
Conversation
Changes preview: |
Add nullable: true to optional fields.
@@ -5078,7 +5109,7 @@ components: | |||
included: | |||
type: array | |||
items: | |||
oneOf: | |||
anyOf: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MatthewKennedy this change breaks displaying of States in includes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll look into it...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@damianlegawiec is this PR code live on: https://api.spreecommerce.org/docs/api-v2/b3A6MzE0Mjc2Nw-get-a-country
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$ref '#...'
nested under an object unnecessarily.IdOrPermalink
into 2 separate references, one dedicatedproductSlug
and another fortaxonPermalink
so examples are more relevant to each.