RSS auto discovery link missing from diary entry lists #4670
Comments
Author: TomH They all seem to have RSS links when I look at them. |
Author: smsm1 It's the orange RSS icon at the bottom of the page on all three of those pages. |
Author: Zverik Ok, I agree that there are RSS links at the bottom. But all RSS readers and browsers expect them to be in meta tag:
So I propose to add this tag where relevant. |
Author: TomH We do add it - that's what the auto_discovery_link_tag line in app/views/diary_entry/list.html.erb does. I think it's being defeated by the caching on the page though, and there's no real way to fix that without reworking the caching completely. It works on my dev instance where there is no caching. Firefox at least long since stopped paying any attention to those tags anyway. |
Author: Zverik I've checked http://www.openstreetmap.org/diary/fr which I've never visited in my life, and there is no RSS auto discovery link. Should it appear after anyone adds another diary entry, or the caching is more complicated than that? |
Author: TomH No the point is that if the page is cached (this is a server side cache) then the view is never run and hence the content_for block doesn't stash the content so when :head is rendered the auto_discovery_link_tag is not there. Basically the cache only stores the actual content of the page and not the side effects like content_for blocks. |
Author: Zverik Is it possible to clear relevant caches when restarting the server, if this would come up anytime soon? Right now it seems like we cannot modify website's pages at all because of caching. Not that it is needed often. |
Author: TomH The caching of this page was removed some time ago, which fixed this. |
Reporter: Zverik
[Submitted to the original trac issue database at 1.18pm, Tuesday, 6th November 2012]
All those pages have no link to their RSS feeds. But they had some time ago. Probably a regression, and needs to be fixed.
The text was updated successfully, but these errors were encountered: