Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upPut the linkset array inside an object #103
Comments
dret
added
the
linkset
label
Aug 24, 2018
This comment has been minimized.
This comment has been minimized.
|
On 2018-08-22 16:38, BigBlueHat wrote:
Using an array as your top-level container cuts of the opportunity to
add information about /all/ the items in the array, it's provenance,
extensions, etc. It greatly limits what's possible.
This opens the door to far more extensibility and clarity than the
current "direct array" format.
thanks for the suggestion! i think i like that idea. @hvdsomp, what are
your thoughts?
|
dret
referenced this issue
Sep 12, 2018
Closed
Use a top-level object (rather than an array) for JSON linksets #105
This comment has been minimized.
This comment has been minimized.
|
Just jumping in to agree with @BigBlueHat 's proposal being a bit more useful - easier to create, manipulate. |
This comment has been minimized.
This comment has been minimized.
|
Yes, I am definitely in support of this too. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
BigBlueHat commentedAug 22, 2018
Using an array as your top-level container cuts of the opportunity to add information about all the items in the array, it's provenance, extensions, etc. It greatly limits what's possible.
Consider reshaping:
[ { "href" : "http://example.com/foo", "anchor" : "http://example.net/bar", "rel" : [ "next" ] }, { "href" : "http://example.com/foo", "anchor" : "http://example.net/bar", "rel" : [ "http://example.net/linkrel" ] } ]to:
{ "linkset": [ { "href" : "http://example.com/foo", "anchor" : "http://example.net/bar", "rel" : [ "next" ] }, { "href" : "http://example.com/foo", "anchor" : "http://example.net/bar", "rel" : [ "http://example.net/linkrel" ] } ] }This opens the door to far more extensibility and clarity than the current "direct array" format.
Cheers!
🎩