Feed for all items

[maiki interi]

Unless I am mistaken, there isn't a feed containing all the items
uploaded to a site. I think that would be useful. =]

[Will Kahn-Greene]

The original url for this bug was ​http://bugs.foocorp.net/issues/701 .

[Sacha]

basically copy the tag_atom_feed but remove the tags in views.py and add the view to /mediagoblin/routing.py

[Sacha]

Got it working, test by going to .../all in lazyserver.

Code is here:
​https://gitorious.org/~chemhacker/mediagoblin/chemhackers-mediagoblin/commits/349_rssfeed

[joar]

The tag_slug is set to empty in the view although it is used to construct the Tag URI (​https://en.wikipedia.org/wiki/Tag_URI) for the feed. It'd be awesome if we'd have an appropriate tag URI there instead :)

Also the feed URL is a bit misleading, /all/atom/ would be a better URL.

[Christopher Allan Webber]

I'll take a look at this today. Those changes Joar suggests should be pretty simple, I think?

[Christopher Allan Webber]

I obviously haven't gotten to this yet. I'll try to get to it soon, but if someone else wants to move on to reviewing it, they're welcome to.

[Will Kahn-Greene]

Did this ever get reviewed?

[Christopher Allan Webber]

This hasn't gotten reviewed, no.

If someone wants to pick it up and review it, that would be helpful. I can't get to it until after the campaign launches.

[spaetz]

I have reviewed the code now. Rebased to latest master (some conflicts in routing.py) and pushed to my git repo at: ​git://gitorious.org/~spaetz/mediagoblin/spaetz-mediagoblin.git branch 349_rssfeed

The code is obviously still Sacha De'Angeli's, thanks for the patch chemhacker!

What is missing:

• No changes to the templates yet, an Atom Feed link needs to be integrated into the homepage where all latest entries are shown (the Atom feed IS advertized in the HEAD part already though).
• Template and atom_feed function are a copy based on the tag_atom_feed, and while I recommend to merge the patch as is, I would also recommend that we look into refactoring the Atom Feed functions into a common core as a second step (otherwise we'll get the same functions for users, tags, years, etc...)

[spaetz]

I just pushed a second commit on top to the branch that cleans up the commit a little further and modifies the "root" template to show the atom feed icon and link as this is where the "most recent media feed" belongs IMHO. The second commit also deletes templates/listings/all.html as contributed by chemhacker as it was unused and not needed.

What remains to be done is to refactor the various feed functions in views.py, consolidating them to one.

[spaetz]

Actually, I believe this issue should be merged into the new lxml-based atom feed branch that I am creating for issue #358. This is a clean and new atom feed branch that avoids duplication as much as possible.

[spaetz]

While the long-term solution will be to move to the lxml-based solution, here is a short term solution, reusing the existing tag_feed view to also provide the all-recent-items feed. See the 2 patches in my branch 349_rssfeed which are ready for review and integration. Reassigning to cwebber to review and OK my branch.

[Christopher Allan Webber]

Merged! (Or more accurately, cherry picked.) Thanks! :)