565362 - [k] Atom feeds for threads and posts

Status: VERIFIED FIXED

(support.mozilla.org :: Forum, task, P2)

Description

[James Socol [:jsocol, :james]]

http://docs.djangoproject.com/en/dev/ref/contrib/syndication/

Here I thought this would take more than 5 minutes.

The correct place for the class definitions is
    apps/forums/feeds.py.

The URLs should look like
    /en-US/forums/forum-slug/feed
    /en-US/forums/forum-slug/1234/feed

Those need to get added apps/forums/urls.py.

Comparing with the examples they give, a few minor differences:
 * Use reverse(), not hard-coded URLs for the link attribute.
 * Localize the title and description attributes (probably _lazy()).
 * We could add get_absolute_url() methods to the new models. It's probably a good idea.
 * Use Atom. It's better.

And test it. That's pretty much it.

Comment 1

[James Socol [:jsocol, :james]]

This is pretty straight forward and will making a great first bug for someone ;)

Comment 2

[Josh Gross (:jgross)]

http://github.com/jsocol/kitsune/commit/bff8ebb8db18c2ea781bb99eba9e0975985cd796

Comment 3

[Stephen Donner [:stephend] Not actively reading bugmail]

Reopening; on https://support-stage-new.mozilla.com/k/en-US/forums/test-forum/3/feed, the links are constructed with example.com, like so:

http://example.com/k/en-US/forums/test-forum/3?page=9#post-368

Comment 4

[James Socol [:jsocol, :james]]

Stephen: does this also affect the forum feeds, eg:

https://support-stage-new.mozilla.com/en-US/forums/test-forum/feed?

(And ftr: you don't need the /k/ for forums anymore.)

Comment 5

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to comment #4)
> Stephen: does this also affect the forum feeds, eg:
> 
> https://support-stage-new.mozilla.com/en-US/forums/test-forum/feed?

Yes, it does
 
> (And ftr: you don't need the /k/ for forums anymore.)

Thanks; copy-and-paste from another bug, I think, or an old URL in my AwesomeBar.

Comment 6

[Ricky Rosario [:rrosario, :r1cky]]

example.com is the default site in the django sites app. Just need to update the database through the admin if you have admin for the sites app enabled.

Comment 7

[James Socol [:jsocol, :james]]

I changed the default site from example.com to support-stage-new.mozilla.com and the feeds now point to the right place.

Just need to be sure we change this on prod.

Comment 8

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED: https://support-stage-new.mozilla.com/en-US/forums/test-forum/243/feed

Comment 9

[Tanay Gavankar [:tgavankar]]

in-testsuite+: Bug 570552