Skip to content
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

Clarify constraints on Param Obj fields (3.1.1 port of #3760) #3796

Merged
merged 2 commits into from May 14, 2024

Conversation

handrews
Copy link
Contributor

@handrews handrews commented May 9, 2024

This makes the relational requirements of content vs schema and associated fields more clear. The mutual exclusivity requirement is moved up to the top of the Fixed Fields section, and each table table is given a subsection clarifying its meaning. This makes it easier to follow the multi-table format, which is unusual within the OAS.

The commentary on example and examples is moved to the section on schema and related parameters as it appears to only apply in that case.

This makes the relational requirements of `content` vs `schema`
and associated fields more clear.  The mutual exclusivity requirement
is moved up to the top of the Fixed Fields section, and each table
table is given a subsection clarifying its meaning.  This makes it
easier to follow the multi-table format, which is unusual within the OAS.

The commentary on `example` and `examples` is moved to the section
on `schema` and related parameters as it appears to only apply
in that case.
@handrews handrews added clarification requests to clarify, but not change, part of the spec param serialization Issues related to parameter and/or header serialization labels May 9, 2024
@handrews handrews added this to the v3.1.1 milestone May 9, 2024
@handrews handrews requested a review from a team May 9, 2024 19:46
versions/3.1.1.md Outdated Show resolved Hide resolved
Thanks to Bellangelo

Co-authored-by: Aggelos Bellos <aggelosbellos7@gmail.com>
Copy link
Contributor

@lornajane lornajane left a comment

Choose a reason for hiding this comment

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

Thanks @handrews !

@ralfhandl ralfhandl merged commit 78d39a1 into OAI:v3.1.1-dev May 14, 2024
1 check passed
@handrews handrews deleted the param-fields-311 branch May 14, 2024 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clarification requests to clarify, but not change, part of the spec param serialization Issues related to parameter and/or header serialization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants