Closed
Bug 565362
Opened 14 years ago
Closed 14 years ago
[k] Atom feeds for threads and posts
Categories
(support.mozilla.org :: Forum, task, P2)
support.mozilla.org
Forum
Tracking
(Not tracked)
VERIFIED
FIXED
2.1
People
(Reporter: jsocol, Assigned: jgross)
References
()
Details
(Whiteboard: [change setting on prod])
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.
Reporter | ||
Comment 1•14 years ago
|
||
This is pretty straight forward and will making a great first bug for someone ;)
Whiteboard: josh+
Reporter | ||
Updated•14 years ago
|
Assignee: nobody → jgross
Assignee | ||
Comment 2•14 years ago
|
||
http://github.com/jsocol/kitsune/commit/bff8ebb8db18c2ea781bb99eba9e0975985cd796
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: josh+
Comment 3•14 years ago
|
||
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
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Updated•14 years ago
|
Reporter | ||
Comment 4•14 years ago
|
||
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•14 years ago
|
||
(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•14 years ago
|
||
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.
Reporter | ||
Comment 7•14 years ago
|
||
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.
Status: REOPENED → RESOLVED
Closed: 14 years ago → 14 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Whiteboard: [change setting on prod]
Comment 8•14 years ago
|
||
Verified FIXED: https://support-stage-new.mozilla.com/en-US/forums/test-forum/243/feed
Status: RESOLVED → VERIFIED
Updated•14 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•