Code

Opened 4 years ago

Last modified 2 years ago

#13110 assigned New feature

Multiple enclosures per feed's item

Reported by: Piaume Owned by: jasonkotenko
Component: contrib.syndication Version: 1.1
Severity: Normal Keywords:
Cc: kotenko@… Triage Stage: Accepted
Has patch: no Needs documentation: yes
Needs tests: yes Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

For now, Feed.item_enclosure_url represents a single enclosure for an item. It should be nice to support many enclosures per item. Something like that:

class ExampleFeed(Feed):
    def item_enclosure_url(self, item):
        return [video.url for video in item.videos]

Attachments (0)

Change History (7)

comment:1 Changed 4 years ago by ericholscher

  • milestone changed from 1.2 to 1.3
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

Feature Request.

comment:2 Changed 4 years ago by russellm

  • milestone 1.3 deleted
  • Triage Stage changed from Unreviewed to Accepted

comment:3 Changed 3 years ago by jasonkotenko

  • Cc kotenko@… added
  • Needs documentation set
  • Needs tests set
  • Owner changed from nobody to jasonkotenko
  • Status changed from new to assigned

comment:4 Changed 3 years ago by jasonkotenko

  • Triage Stage changed from Accepted to Design decision needed

It appears that RSS only officially supports one enclosure per item {1}, and Atom supports multiple. Since RSS is the default feed, actually the use case presented in the bug report is invalid. The question then becomes, should we support multiple enclosures for Atom? As of now the enclosure is handled in the same general way in both types of feeds.

Changing to "Design Decision Needed" before I do anything else... Please let me know if that was the wrong thing to do in this situation, as I'm pretty new to the Django community.

{1} - http://www.rssboard.org/rss-profile#element-channel-item-enclosure

comment:5 Changed 3 years ago by lukeplant

  • Type set to New feature

comment:6 Changed 3 years ago by lukeplant

  • Severity set to Normal

comment:7 Changed 2 years ago by aaugustin

  • Easy pickings unset
  • Triage Stage changed from Design decision needed to Accepted
  • UI/UX unset

It should be possible to accept either a string or a list of string, and to validate that there's only one enclosure for RSS feeds.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as assigned
The owner will be changed from jasonkotenko to anonymous. Next status will be 'assigned'
The ticket will be disowned. Next status will be 'new'
as The resolution will be set. Next status will be 'closed'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.