Adding google news sitemap support to django.contrib.sitemaps

[Lakin Wecker]

​Google News Sitemaps are very similar to normal sitemaps in that they build off of the sitemaps protocol. They add a new namespace to the xml file, along with some added news-specific attributes to eatch url.

This patch adds support for generation of these news sitemaps to the django.contrib.sitemaps application.

There were no tests for this package - so I have not added any news tests. I have however, made an attempt to update the documentation for sitemaps.

[anonymous]

It looks like you left a bit of hard-coded data on line 14 of django/contrib/sitemaps/templates/news_sitemap.xml

{% if url.keywords %}<news:keywords>Business, Mergers, Acquisitions</news:keywords>{% endif %} 

[Lakin Wecker]

There are a couple of small bugs that i discovered this morning - working through those now.

[Lakin Wecker]

Fixed the bugs.

[Lakin Wecker]

Ok, those bugs have been ferretted out.

[ChrisL]

Ticket patch not updated.

I'm implementing this patch and can see the <news:keywords> content is still in place. Should read:

{% if url.keywords %}<news:keywords>

{% for keyword in keywords %}{{ url.keyword }}{% if not forloop.last %}, {% endif %}{% endfor %}

</news:keywords>{% endif %}

can't get the news sitemap content to pass through to the template correctly, likely this was part of the bugs that were cleaned up?

Please post the up to date patch.

[Timothée Peignier <timothee.peignier@…>]

There also a mobile sitemap format existing, see : ​http://www.google.com/support/webmasters/bin/answer.py?answer=34648&hl=en

[Jannis Leidel]

In [16474]:

Fixed #10907, #14190 and #15829 -- Pass item to sitemaps template to allow further customization like Google News enabled sitemaps. Thanks, manfre and lakinwecker.