search

innoq / statuses

statuses
Apache License 2.0
13 stars 14 forks source link

Atom feed doesn't contain reference links #72

Closed aheusingfeld closed 9 years ago

aheusingfeld commented 9 years ago

The Atom feed doesn't contain links to

mvitz commented 9 years ago

The author is already linked by entry/author/uri.

The rel-attribute of a link-element has only a limited set of values (see http://www.iana.org/assignments/link-relations/link-relations.xhtml).

Conversation can be mapped to related. Unfortunately there is no direct mapping for in-reply-to. Only an extension element exists (see: http://tools.ietf.org/html/rfc4685#section-3). I'm not sure if any feed reader can handle such an extension.

aheusingfeld commented 9 years ago

Unfortunately there is no direct mapping for in-reply-to.

There is a "rel=prev" which should contain the link to the "in-reply-to" post.

mvitz commented 9 years ago

I would expect with prev the previous entry from the timeline (which doesn't have to be the one this entry replied to).

aheusingfeld commented 9 years ago

Maybe you should adjust your expectation to the specification?! :)

Indicates that the link's context is a part of a series, and that the previous in the series is the link target.

But seriously: There is no semantic link between regular entries in the "timeline". Therefore we shouldn't assume a technical link to be there. In other words: IMHO "the timeline" is not a linked series of posts!

mvitz commented 9 years ago

Ok. Thanks for enlightening me.

If you are fine with 'rel=related' for conversations I will start implementing this.

aheusingfeld commented 9 years ago

Thanks for enlightening me.

Well, most often it's good to discuss opposing opinions because we're all fallible! So, thank you for taking the time!

If you are fine with 'rel=related' for conversations I will start implementing this.

To be honest: My work on https://github.com/innoq/statuses/issues/19#issuecomment-60305389 will actually remove the conversation view and instead introduce a redirect to the update/ post itself. Therefore I wouldn't take the effort to also implement "rel=related" anymore if I were you.

mvitz commented 9 years ago

As long as there is a conversation view I will add the link. ;-) But thanks for your warning.