[Django] #31822: Add support for item_comments dynamic attribute

20 views
Skip to first unread message

Django

unread,
Jul 23, 2020, 9:49:17 AM7/23/20
to django-...@googlegroups.com
#31822: Add support for item_comments dynamic attribute
-------------------------------------+-------------------------------------
Reporter: Viktor | Owner: Viktor Garske
Garske |
Type: New | Status: assigned
feature |
Component: | Version: 3.0
contrib.syndication |
Severity: Normal | Keywords:
Triage Stage: | Has patch: 0
Unreviewed |
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 1
UI/UX: 0 |
-------------------------------------+-------------------------------------
Add ''comments'' argument to {{{ feed.add_item() }}} in syndication.views
so that {{{ item_comments }}} can be defined directly without having to
take the detour via {{{ item_extra_kwargs }}}.

Additionally, ''comments'' is already explicitly mentioned in the
feedparser, but not implemented in the view.

--
Ticket URL: <https://code.djangoproject.com/ticket/31822>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Jul 23, 2020, 11:04:22 AM7/23/20
to django-...@googlegroups.com
#31822: Add support for item_comments dynamic attribute
-------------------------------------+-------------------------------------
Reporter: Viktor Garske | Owner: Viktor
| Garske
Type: New feature | Status: assigned
Component: contrib.syndication | Version: 3.0
Severity: Normal | Resolution:
Keywords: | Triage Stage:
| Unreviewed
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Viktor Garske):

* has_patch: 0 => 1


--
Ticket URL: <https://code.djangoproject.com/ticket/31822#comment:1>

Django

unread,
Jul 23, 2020, 11:23:48 AM7/23/20
to django-...@googlegroups.com
#31822: Add support for item_comments dynamic attribute
-------------------------------------+-------------------------------------
Reporter: Viktor Garske | Owner: Viktor
| Garske
Type: New feature | Status: assigned
Component: contrib.syndication | Version: master

Severity: Normal | Resolution:
Keywords: | Triage Stage:
| Unreviewed
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Viktor Garske):

* version: 3.0 => master


--
Ticket URL: <https://code.djangoproject.com/ticket/31822#comment:2>

Django

unread,
Jul 24, 2020, 1:27:46 AM7/24/20
to django-...@googlegroups.com
#31822: Add support for item_comments to syndication framework

-------------------------------------+-------------------------------------
Reporter: Viktor Garske | Owner: Viktor
| Garske
Type: New feature | Status: assigned
Component: contrib.syndication | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by felixxm):

* stage: Unreviewed => Accepted


Comment:

Sounds reasonable, see
[http://validator.w3.org/feed/docs/rss2.html#hrelementsOfLtitemgt RSS
2.0].

[https://github.com/django/django/pull/13230 PR]

--
Ticket URL: <https://code.djangoproject.com/ticket/31822#comment:3>

Django

unread,
Jul 30, 2020, 1:43:10 AM7/30/20
to django-...@googlegroups.com
#31822: Add support for item_comments to syndication framework
-------------------------------------+-------------------------------------
Reporter: Viktor Garske | Owner: Viktor
| Garske
Type: New feature | Status: assigned
Component: contrib.syndication | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by felixxm):

* stage: Accepted => Ready for checkin


--
Ticket URL: <https://code.djangoproject.com/ticket/31822#comment:4>

Django

unread,
Jul 30, 2020, 2:16:19 AM7/30/20
to django-...@googlegroups.com
#31822: Add support for item_comments to syndication framework
-------------------------------------+-------------------------------------
Reporter: Viktor Garske | Owner: Viktor
| Garske
Type: New feature | Status: closed
Component: contrib.syndication | Version: master
Severity: Normal | Resolution: fixed

Keywords: | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Mariusz Felisiak <felisiak.mariusz@…>):

* status: assigned => closed
* resolution: => fixed


Comment:

In [changeset:"1173db4a16bb2938ba62a6cd50372a76a7f9e05f" 1173db4a]:
{{{
#!CommitTicketReference repository=""
revision="1173db4a16bb2938ba62a6cd50372a76a7f9e05f"
Fixed #31822 -- Added support for comments URL per feed item.

The item_comments hook returns a comments URL which is then used by the
feed builder.
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/31822#comment:5>

Reply all
Reply to author
Forward
0 new messages