The feed validator says we need an atom:self link

RESOLVED WONTFIX

Status

addons.mozilla.org Graveyard
Public Pages
P5
trivial
RESOLVED WONTFIX
9 years ago
2 years ago

People

(Reporter: jbalogh, Assigned: jbalogh)

Tracking

unspecified
4.x (triaged)

Details

(URL)

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

9 years ago
Created attachment 375911 [details] [diff] [review]
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.
Attachment #375911 - Flags: review?(clouserw)
Attachment #375911 - Flags: review?(clouserw) → review-
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.
(Assignee)

Comment 2

9 years ago
Created attachment 376121 [details] [diff] [review]
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!
Attachment #375911 - Attachment is obsolete: true
Attachment #376121 - Flags: review?(clouserw)
(Assignee)

Comment 3

9 years ago
Created attachment 376129 [details] [diff] [review]
adding possibility for atom:self link in rss

Uploading the right patch this time.
Attachment #376121 - Attachment is obsolete: true
Attachment #376129 - Flags: review?(clouserw)
Attachment #376121 - Flags: review?(clouserw)

Updated

8 years ago
Severity: normal → trivial
Priority: -- → P5
Target Milestone: --- → 4.x (triaged)
(Assignee)

Comment 4

7 years ago
RESOLVED => EXPIRED
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → WONTFIX
Attachment #376129 - Flags: review?(clouserw)
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.