Re: Internet-Draft: JSON Hypertext Application Language

Hey Enrique thanks for your feedback, comments in line

On Fri, Jun 8, 2012 at 2:03 PM, Enrique Amodeo <[email protected]> wrote:
> I like it! Simple and useful. I have some comments.
> I'd change the redaction a bit in section 4.1: "It is an object whose
> names are link relation types" should be "It is an object whose
> *property* names are link relation types". The same thing in 4.2.

thanks, will amend

> Appart for these, I'd remove the "hreflang" link attribute, what if a
> resource supports several languages? The language stuff seems better
> addressed using HTTP headers rather than a link attribute.

This is optional and intended as a 'hint' as this is how its defined
in the Web Linking RFC that's referenced.

I was actually considering introducing all of the remaining link
params introduced by Web Linking - I take it you would be against that
decision?

> I'd remove
> the "self" link since it is redundant. The resource consumer already
> knows the resource URIs.

That may be true for the root resource, but not for the embedded
resources. I think it's 'tidier' to include it for all resource
objects as it is consistent and lends itself better to situations
where resources are included as partials within other documents.

Cheers,
M