-
Notifications
You must be signed in to change notification settings - Fork 13
Updates to OpenAPI 3.2 Guide #412
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
base: main
Are you sure you want to change the base?
Conversation
❌ Deploy Preview for bump-content-hub failed. Why did it fail? →
|
b1461d8 to
0918613
Compare
0918613 to
3e46343
Compare
| - Cookie Parameters - `in: cookie` | ||
| - OpenAPI 3.2 also added Querystring Parameters `in: querystring` | ||
|
|
||
| That new addition is a little confusing as the names are so similar, but we'll cover that in a bit. |
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'm ok with this, even though it might look a bit judgemental?
Real question now: Should the bump.sh guide be opinionated and “push” users to use only a subset of the most relevant features and try to disregard other features that seem less useful/clear to us?
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.
It's not an opinion about the quality of the naming, its an observation that a user new to the concepts may be confused by their similarity.
We are not suggesting somebody should use one or the other, just noting that confusion and giving them confidence it will be explained further down.
Fixes #400.
Several updates for the OAS 3.2 guide.
in: querystringas per OAS 3.2: querystring #400