innoq / statuses

statuses
Apache License 2.0
13 stars 14 forks source link

Atom: Id should be url #5

Closed martinei closed 11 years ago

martinei commented 11 years ago

From the Validator: line 1, column 88: id must be a full and valid URL: some-id [help]

... >innoQ Status updates</title><id>some-id</id><updated>2012-11-23T05:48:0 ...
                                             ^

line 1, column 364: id must be a full and valid URL: 50 (25 occurrences) [help]

... est</title><summary>fsdf</summary><id>50</id><published>2012-11-23T05:48 ...

My Problem: Do we already have some kind ob base-url funciton?

stilkov commented 11 years ago

Yes, see here.

martinei commented 11 years ago

Thanks. I will than approach this task (later...)

stilkov commented 11 years ago

Added the feed URI as the ID, didn't bother to validate it using the Feed validator because I'm too lazy.

martinei commented 11 years ago

It validates. However Vienna links to https://internal2.innoq.com/statuses/internal2.innoq.com Links to the individual feeds are fine. I'll try to find out why

stilkov commented 11 years ago

Seems to be fixed AFAICT.