491616 - The feed validator says we need an atom:self link

Status: RESOLVED WONTFIX

(addons.mozilla.org Graveyard :: Public Pages, defect, P5)

Description

[Jeff Balogh (:jbalogh)]

Attachment: adding an atom:self link

It's dubious whether we actually need this, and the way I'm doing it is probably wrong.  Feel free to wontfix.

The patch will only work if we always use ?format=rss for feeds.

Comment 1

[Wil Clouser [:clouserw]]

Comment on attachment 375911 [details] [diff] [review]
adding an atom:self link

I think adding the link is fine, but the URL you're using won't work.  A lot of times we just have /format:rss as part of the URL and sometimes we have additional ?sort= stuff on the end.

Comment 2

[Jeff Balogh (:jbalogh)]

Attachment: adding possibility for atom:self link in rss

This version depends on the feed creator to define $atom_self, but at least we'll have the possibility!

Comment 3

[Jeff Balogh (:jbalogh)]

Attachment: adding possibility for atom:self link in rss

Uploading the right patch this time.

Comment 4

[Jeff Balogh (:jbalogh)]

RESOLVED => EXPIRED