-
Notifications
You must be signed in to change notification settings - Fork 9.1k
Hyperlink support in Swagger #198
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
+1. Happy to contribute to shaping this one |
I would suggest starting with how you would want both the ui and clients to take advantage of link support |
Sounds like a plan. @mattbishop let's chat |
There has been no action on this since Nov 28. If we want to use Swagger as a description language, this is a deal-breaker. |
@ashokmalhotra - unlike the tooling, the spec doesn't change too often. We've released 2.0 back in September so you shouldn't expect seeing a new version this soon. |
Is there enough interest to add Link support to the spec? |
Really too early to say. There's definitely interest in it and we'll discuss it when the time comes. And no, I cannot give you a rough date. |
Parent: #555. |
We are currently using x-links in swagger, and the content of x-links is pretty much the json hyper schema's links. |
How are hyperlinks currently implemented? |
@awwright In v3 operations can be related via the links object (some description here https://www.openapis.org/blog/2016/11/14/tdc-protocol-and-payload-explaining-the-3-0-spec-part-4 ). However this is purely a design time concept. |
So what do we think about this one? Is links enough, are we leaving the rest to the hypermedia formats of the world, or is there ever likely to be further work on hypermedia. |
The only other work I see is the ability to define path Item Objects that don't have a fixed URL. And we sort of are doing that with making Path Item Objects components. I don't think there is anything novel in this issue to warrant keeping it open. |
It will be nice to provide support for link relations in swagger.
The text was updated successfully, but these errors were encountered: