Extended category support for feeds

85 views
Skip to first unread message

Pavlo Kapyshin

unread,
Jan 30, 2017, 7:02:53 PM1/30/17
to django-d...@googlegroups.com
Hello,

I’ve made a patch that adds ability to specify “scheme” (or “domain”, for RSS)
and “label” on <category> elements in RSS and Atom1 feeds. Could you please
look at it and maybe suggest some improvements?

https://github.com/PavloKapyshin/django/commit/9c03f13d2442f8dacc98514e1f5d1a332bc1dfc1

There is relevant ticket: https://code.djangoproject.com/ticket/27792

Florian Apolloner

unread,
Jan 31, 2017, 6:05:33 AM1/31/17
to Django developers (Contributions to Django itself)
Hi,

there is no need to write to the mailing list to inform us of any patches. Please just open a pull request and link it to the ticket in question.

Cheers,
Florian

Florian Apolloner

unread,
Jan 31, 2017, 6:07:55 AM1/31/17
to Django developers (Contributions to Django itself)
Ah sorry, didn't see that Tim suggested that you should find other reviewers. Either way, please open a pull request, since that makes it easier to commit on commits.

qtake...@gmail.com

unread,
Jan 31, 2017, 7:08:19 AM1/31/17
to Django developers (Contributions to Django itself)

Looks good. I use syndication in few projects and I like that this change does not break my code.



On Tuesday, January 31, 2017 at 1:02:53 AM UTC+1, Pavlo Kapyshin wrote:

Pavlo Kapyshin

unread,
Jan 31, 2017, 10:08:55 AM1/31/17
to Django developers (Contributions to Django itself)
Reply all
Reply to author
Forward
0 new messages