search

getpelican / feedgenerator

Standalone version of Django's feedgenerator module
BSD 3-Clause "New" or "Revised" License
52 stars 23 forks source link

Always set the 'updated' element of an 'entry' in Atom Feeds #7

Closed Scheirle closed 8 years ago

Scheirle commented 8 years ago

Acording to the RFC4287[1] this element is mandatory.

Dependent on what is set by the user it gets set to updateddate, pubdate or the current date time

[1] https://tools.ietf.org/html/rfc4287#section-4.1.2

justinmayer commented 8 years ago

Thanks for the contribution, @Scheirle.

@eevee: As the author of many of the Atom-related enhancements (see #3), what do you think of this change? Have a moment to review and provide any feedback you may have?

eevee commented 8 years ago

Seems reasonable to me!

I'm surprised some sort of date isn't required — a feed isn't very useful without timestamps.

justinmayer commented 8 years ago

Thanks, Eevee. Much appreciated!

ingwinlu commented 8 years ago

@justinmayer feedgenerator should be bumped to 1.9 asap. currently pelican is pulling in 1.8 which isn't conforming to the rfc as mentioned above.

justinmayer commented 8 years ago

FeedGenerator 1.9 has been published to PyPI.