http://drupal.org/cvs shows a feed icon at the bottom of the page. It would be nice if the HTML source would contain a <link rel="alternate"> tag to make feed discovery easier.

CommentFileSizeAuthor
#1 396212_cvs_feed_link.patch611 bytesgreggles

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Project: Drupal.org site moderators » CVS integration
Version: » 6.x-1.x-dev
Component: Redesign » Code
Status: Active » Needs review
StatusFileSize
new 396212_cvs_feed_link.patch611 bytes

Updating to appropriate project and a patch that might get this right.

It is untested and may end up providing the feed icon twice.

jpetso’s picture

jpetso commented
Status: Needs review » Reviewed & tested by the community

r0xx0rz!!! I always wanted to have this, but I had no idea it's *that* simple :D
Ported and committed to Version Control API's Commit Log, thanks!

(Seems to work properly, too. At least the straight port is.)

dww’s picture

dww
we/he/they
commented
Status: Reviewed & tested by the community » Needs work

This does indeed provide the feed icon twice:
http://d6.drupal.org/cvs
But, I can remove the feed_icon call before I commit...

dww’s picture

dww
we/he/they
commented
Status: Needs work » Fixed

Removed theme('feed_icon'), tested, and committed to HEAD. This will go out on d.o once there are some other useful changes and I need to deploy a new version of cvs.module.

Thanks!
-Derek

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

Cool, thanks for testing and committing this, dww. It's a little hard (at least for me) to setup a local environment to test out stuff like this, so I appreciate you doing that.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.