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

Move complex Sec Req example to appendix F #4090

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

handrews
Copy link
Member

Inspired by @mikekistler 's correct observation that this section was confusingly out-of-place.

The extended example with a multi-document OAD and a Security Requirement in the referenced document
did to fit well where it was, and there wasn't room in the Resolving Implicit Connections area. So this moves it to an Appendix linked from both locations.

@handrews handrews requested a review from a team as a code owner September 12, 2024 16:35
The extended example with a multi-document OAD and
a Security Requirement in the referenced document
did to fit well where it was, and there wasn't room in the
Resolving Implicit Connections area.  So this moves
it to an Appendix linked from both locations.
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! 👍

Thanks for this change!

@handrews handrews added this to the v3.0.4 milestone Sep 12, 2024
@handrews handrews added the re-use: ref/id resolution how $ref, operationId, or anything else is resolved label Sep 12, 2024
@lornajane lornajane merged commit 2eb4ffe into OAI:v3.0.4-dev Sep 12, 2024
2 checks passed
@handrews handrews deleted the sec-req-304 branch September 12, 2024 20:03
@ralfhandl ralfhandl mentioned this pull request Sep 30, 2024
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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