search

getpelican / feedgenerator

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

Remove unnecessary atom elements. #1

Closed SoftlySplinter closed 10 years ago

SoftlySplinter commented 10 years ago

This should fix getpelican/pelican#1089

justinmayer commented 10 years ago

Does this also remove said elements from the Atom feed? If so, that might be a problem.

I realize it would involve extra work, but perhaps you might consider adding a test.atom file to ensure that behaviors specific to each format are handled appropriately?

SoftlySplinter commented 10 years ago

Good point, I shall add some tests in and get back to you.

SoftlySplinter commented 10 years ago

Doesn't seem to affect Atom feed generation, I've added a test in the commit above.

Hope that helps.

justinmayer commented 10 years ago

It helps a great deal. Many thanks!