Skip to content

Fix Document vs Description terminology (3.1.1, port of #4100) #4131

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

Merged
merged 11 commits into from
Oct 14, 2024

Conversation

handrews
Copy link
Member

I think this is correct (and includes the last remaining "suggestion" about the Server Object url field wording), but please review carefully. I still need to re-review it myself, as resolving conflicts and adjusting for 3.1.1 was a bit complex in spots.

Note that #4130 does not need to be ported as it is already handled by having the two distinct "Relative References in..." (URI vs URL) sections. Although those sections assume the split documented in #4130 that we said was ambiguous in 3.0.

@handrews handrews added re-use: ref/id resolution how $ref, operationId, or anything else is resolved editorial Wording and stylistic issues labels Oct 10, 2024
@handrews handrews added this to the v3.1.1 milestone Oct 10, 2024
@handrews handrews requested a review from a team as a code owner October 10, 2024 20:09
Copy link
Contributor

@ralfhandl ralfhandl left a comment

Choose a reason for hiding this comment

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

The same three nits as in #4100

@ralfhandl ralfhandl requested a review from a team October 11, 2024 08:00
Copy link
Contributor

@mikekistler mikekistler left a comment

Choose a reason for hiding this comment

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

Looks good! 👍

@ralfhandl ralfhandl merged commit bf46ac7 into OAI:v3.1.1-dev Oct 14, 2024
2 checks passed
@handrews handrews deleted the desc-v-doc-311 branch October 14, 2024 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editorial Wording and stylistic issues re-use: ref/id resolution how $ref, operationId, or anything else is resolved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants