-
Notifications
You must be signed in to change notification settings - Fork 9.1k
description
/summary
for Paths Object
#3722
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
Comments
@ralfhandl can you assign this one to me please? |
I suggest not doing this and instead let Moonwalk solve this problem. RationaleThe original issue states that the Info Object may not be a good place to describe both paths and webhooks:
I don't see this as a problem because We cannot add We could theoretically add Also adding these fields only for the paths would kind of skew |
I don't have a strong opinion on this. I was simply volunteering to do the grunt work. And in the meantime, maybe @handrews can chime in on additional context/rationale for this. |
I also suggest that we do not implement a change like this in 3.x (thanks @ralfhandl for the good reasoning). If the paths and webhooks don't belong together then separate them - or use heading markdown in the info section to include information about each part. I will close this issue now, but please feel free to request it be re-opened if we're missing something. |
Discussed in #3532
Originally posted by kentbulza January 28, 2024
This seems like an obvious gap to me, but I want to see what were the philosophical reasons for it. Perhaps because the webhooks object is newer, it just wasn't as obvious at first that the info object can be awkward? The issue is there may be very different global(esque) descriptive text for all the paths vs. all the webhooks.
The concern here is about describing Paths and Webhooks separately. It's possible to put each in separate documents with separate Info Objects, but it is not possible to reference an entire Paths Object (or the entire contents of the
webhooks
field, which isn't a named Object in 3.1). I could see solving this by either adding these fields (they cannot be confused with path templates, although that doesn't help with the object underwebhooks
) or the much larger change of allowing referencing groups of Paths (which might be deemed too big to do prior to MoonwalkThe text was updated successfully, but these errors were encountered: