Skip to content

"description" in the LDO #325

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

Closed
handrews opened this issue May 28, 2017 · 4 comments
Closed

"description" in the LDO #325

handrews opened this issue May 28, 2017 · 4 comments

Comments

@handrews
Copy link
Contributor

The LDO has "title" but not "description". When you think about the LDO in RFC 5988 or similar terms, this makes sense. But LDOs can easily support much more information than most link descriptions.

In my experience, most schema authors expect "description" to be present in the LDO as well and used by documentation tools. That doesn't violate any aspect of JSON Hyper-Schema, but I think it should be official as it is the obvious expected behavior within a JSON Schema (rather than RFC 5988) context.

@handrews handrews added this to the draft-07 (wright-*-02) milestone May 28, 2017
@handrews
Copy link
Contributor Author

handrews commented Aug 19, 2017

Paging some hypermedia users (off the top of my head, apologies for missing anyone) @awwright @geemus @seagreen @dret @dlax any thoughts on this? I see @jdesrosiers gave it a thumbs up. This should be a pretty simple yes or no, I hope.

We use this extensively in Cloudflare's documentation tool. I'd suggest the API Doc vocabulary, except this exactly parallels "description" in the validation vocabulary so it seems to belong in the spec proper.

@dret
Copy link
Contributor

dret commented Aug 19, 2017 via email

handrews added a commit to handrews/json-schema-spec that referenced this issue Aug 20, 2017
This addresses issue json-schema-org#325.  It adds the description keyword,
analogous to "description" in validation schemas.  In RFC 5988bis
parlance, "description" is an extension target attribute.
@geemus
Copy link
Collaborator

geemus commented Aug 21, 2017

LGTM

@handrews
Copy link
Contributor Author

handrews commented Sep 2, 2017

Added with #353.

@handrews handrews closed this as completed Sep 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants