Posted by Bèr Kessels on June 23, 2006 at 8:48am
| Project: | Project |
| Version: | x.y.z |
| Component: | User interface |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (duplicate) |
Issue Summary
Often requested, simple to implement: An easy and quick way to track a project in RSS. Patch adds that.
After the patch the line goes like
» View CVS messages (RSS).
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| addCVS_RSSlink.patch.txt | 641 bytes | Ignored: Check issue status. | None | None |
Comments
#1
Didn't work for me
your $query has a "?" in it., taking out the ?, everything works fine.
#2
removed the ?
#3
seriously man, you need to actually test this
replaced the "." with "," before the query.
#4
latest patch seems reasonable upon simple inspection. however, i'm currently on vacation until july 16th, with no good means to actually test anything like this. i refuse to commit changes to the project module that i can't test, so this will either have to wait, or nedjo/dries will have to test and commit it.
furthermore, there's active effort towards making the project module more RCS-independent (http://drupal.org/node/70125) and this particular patch is a step away from that goal. this is just a minor addition to an existing problem, so i'm not going to refuse it on those grounds. however, in general, i'd prefer if new patches that touch CVS stuff do it in a way that moves CVS-specific functionality out of project and into cvslog, where it belongs. for example, adding this link could be done via hook_nodeapi() in cvs.module...
thanks,
-derek
#5
This is a duplicate of #31994: Add a link to CVS RSS feeds from project node page.