[Django] #27792: Extended category support for Atom and RSS feeds
Django
-------------------------------------+-------------------------------------
Reporter: Pavlo | Owner: nobody
Kapyshin |
Type: New | Status: new
feature |
Component: | Version: 1.10
contrib.syndication |
Severity: Normal | Keywords: feedgenerator
Triage Stage: | Has patch: 1
Unreviewed |
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-------------------------------------+-------------------------------------
I’m working on category (label, scheme/domain) attributes support for Atom
and RSS. I have a patch for Atom
(https://github.com/PavloKapyshin/django/tree/feeds_category_object), and
want to ask what's the best way to implement same functionality for RSS
feeds.
RSS domain maps fairly well to Atom's scheme.
Question is: for RSS, should category label be taken from Atom namespace
(so there'll be an attribute) or just put inside <category> instead of
term, if available?
--
Ticket URL: <https://code.djangoproject.com/ticket/27792>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
Django
-------------------------------------+-------------------------------------
| Kapyshin
Type: New feature | Status: assigned
Component: contrib.syndication | Version: 1.10
Severity: Normal | Resolution:
Keywords: feedgenerator | Triage Stage:
| Unreviewed
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
* owner: nobody => Pavlo Kapyshin
* status: new => assigned
Old description:
> I’m working on category (label, scheme/domain) attributes support for
> Atom and RSS. I have a patch for Atom
> (https://github.com/PavloKapyshin/django/tree/feeds_category_object), and
> want to ask what's the best way to implement same functionality for RSS
> feeds.
>
> RSS domain maps fairly well to Atom's scheme.
>
> Question is: for RSS, should category label be taken from Atom namespace
> (so there'll be an attribute) or just put inside <category> instead of
> term, if available?
New description:
I’m working on category (label, scheme/domain) attributes support for Atom
and RSS. I have a patch for Atom
(https://github.com/PavloKapyshin/django/tree/feeds_category_object), and
want to ask what's the best way to implement same functionality for RSS
feeds.
RSS domain maps fairly well to Atom's scheme.
Question is: for RSS, should category label be taken from Atom namespace
(so there'll be an attribute) or just put inside <category></category>
instead of term, if available?
--
--
Ticket URL: <https://code.djangoproject.com/ticket/27792#comment:1>
Django
-------------------------------------+-------------------------------------
| Kapyshin
Type: New feature | Status: assigned
Severity: Normal | Resolution:
Keywords: feedgenerator | Triage Stage:
| Unreviewed
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
* version: 1.10 => master
--
Ticket URL: <https://code.djangoproject.com/ticket/27792#comment:2>
Django
-------------------------------------+-------------------------------------
| Kapyshin
Type: New feature | Status: assigned
Component: contrib.syndication | Version: master
Severity: Normal | Resolution:
Keywords: feedgenerator | Triage Stage:
| Unreviewed
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
* needs_docs: 0 => 1
Old description:
> I’m working on category (label, scheme/domain) attributes support for
> Atom and RSS. I have a patch for Atom
> (https://github.com/PavloKapyshin/django/tree/feeds_category_object), and
> want to ask what's the best way to implement same functionality for RSS
> feeds.
>
> RSS domain maps fairly well to Atom's scheme.
>
> Question is: for RSS, should category label be taken from Atom namespace
> (so there'll be an attribute) or just put inside <category></category>
> instead of term, if available?
New description:
I’m working on category (label, scheme/domain) attributes support for Atom
and RSS. I have a patch for all three feed types:
https://github.com/PavloKapyshin/django/tree/feeds_category_object
Next step is to work on docs for this.
--
--
Ticket URL: <https://code.djangoproject.com/ticket/27792#comment:3>
Django
-------------------------------------+-------------------------------------
| Kapyshin
Type: New feature | Status: assigned
Component: contrib.syndication | Version: master
Severity: Normal | Resolution:
Keywords: feedgenerator | Triage Stage:
| Unreviewed
Has patch: 1 | Needs documentation: 1
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
* Attachment "0001-Refs-27792-Added-extended-category-support-for-
feeds.patch" added.
Django
-------------------------------------+-------------------------------------
| Kapyshin
Type: New feature | Status: assigned
Component: contrib.syndication | Version: master
Severity: Normal | Resolution:
Keywords: feedgenerator | Triage Stage:
| Unreviewed
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
* needs_docs: 1 => 0
--
Ticket URL: <https://code.djangoproject.com/ticket/27792#comment:4>
Django
-------------------------------------+-------------------------------------
| Kapyshin
Type: New feature | Status: assigned
Component: contrib.syndication | Version: master
Severity: Normal | Resolution:
Keywords: feedgenerator | Triage Stage:
| Unreviewed
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Old description:
> I’m working on category (label, scheme/domain) attributes support for
> Atom and RSS. I have a patch for all three feed types:
> https://github.com/PavloKapyshin/django/tree/feeds_category_object
>
> Next step is to work on docs for this.
New description:
This patch adds category (label, scheme/domain) attributes support for
Atom and RSS feeds.
--
--
Ticket URL: <https://code.djangoproject.com/ticket/27792#comment:5>
Django
-------------------------------------+-------------------------------------
| Kapyshin
Type: New feature | Status: assigned
Component: contrib.syndication | Version: master
Severity: Normal | Resolution:
Keywords: feedgenerator | Triage Stage:
| Unreviewed
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Old description:
> This patch adds category (label, scheme/domain) attributes support for
> Atom and RSS feeds.
New description:
This patch adds category (label, scheme/domain) attributes support for
Atom and RSS feeds.
Attachment is outdated, see
https://github.com/PavloKapyshin/django/commit/9c03f13d2442f8dacc98514e1f5d1a332bc1dfc1
--
--
Ticket URL: <https://code.djangoproject.com/ticket/27792#comment:6>
Django
-------------------------------------+-------------------------------------
| Kapyshin
Type: New feature | Status: assigned
Component: contrib.syndication | Version: master
Severity: Normal | Resolution:
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
* stage: Unreviewed => Accepted
Old description:
> This patch adds category (label, scheme/domain) attributes support for
> Atom and RSS feeds.
>
> Attachment is outdated, see
> https://github.com/PavloKapyshin/django/commit/9c03f13d2442f8dacc98514e1f5d1a332bc1dfc1
New description:
This patch adds category (label, scheme/domain) attributes support for
Atom and RSS feeds.
See
https://github.com/PavloKapyshin/django/commit/9c03f13d2442f8dacc98514e1f5d1a332bc1dfc1
--
Comment:
I'm not a user of the syndication framework but this looks okay at first
glance. If you could find another syndication user to review it, that
would be really helpful. Please send a pull request when the patch is
conforming to the PatchReviewChecklist.
--
Ticket URL: <https://code.djangoproject.com/ticket/27792#comment:7>
Django
-------------------------------------+-------------------------------------
| Kapyshin
Type: New feature | Status: assigned
Component: contrib.syndication | Version: master
Severity: Normal | Resolution:
Keywords: feedgenerator | Triage Stage: Accepted
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Comment (by Pavlo Kapyshin):
https://github.com/django/django/pull/8001
--
Ticket URL: <https://code.djangoproject.com/ticket/27792#comment:8>
Django
-------------------------------------+-------------------------------------
| Kapyshin
Type: New feature | Status: assigned
Component: contrib.syndication | Version: master
Severity: Normal | Resolution:
Keywords: feedgenerator | Triage Stage: Accepted
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Comment (by Pavlo Kapyshin):
Also there is topic at django-developers maillist:
https://groups.google.com/forum/#!topic/django-developers/H8fcZ6dq_0s
--
Ticket URL: <https://code.djangoproject.com/ticket/27792#comment:9>
Django
-------------------------------------+-------------------------------------
| Kapyshin
Component: contrib.syndication | Version: master
Severity: Normal | Resolution: wontfix
Keywords: feedgenerator | Triage Stage: Accepted
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
* status: assigned => closed
* resolution: => wontfix
Comment:
The reporter and patch author has withdrawn this ticket, "I think feed
generation (syndication) should live in separate specialized library, that
is not tied to particular web framework or ORM."
--
Ticket URL: <https://code.djangoproject.com/ticket/27792#comment:10>