Originally by aclight.
I'd love a review on this before committing. It also needs a 5.x port, but that shouldn't be bad.
| Comment | File | Size | Author |
|---|---|---|---|
| comment_notify_6x_private.patch | 4.67 KB | greggles |
Originally by aclight.
I'd love a review on this before committing. It also needs a 5.x port, but that shouldn't be bad.
| Comment | File | Size | Author |
|---|---|---|---|
| comment_notify_6x_private.patch | 4.67 KB | greggles |
Comments
Comment #1
scottrigby@greggles: if this is still open I'll take a look soon... btw (before digging in), does this cover the case where users can create subscriptions to content types they don't have access to? Anyway, i'll check this out - but curious about this case
Comment #2
scottrigbyoops - greggles - scratch my comment above (it's way too early in the morning - I was actually looking for this case in the notifications module... I'm using both on the same site :p )
Comment #3
gregglesNow committed - http://drupal.org/cvs?commit=277504
Comment #4
gregglesThis module is no longer actively developed for Drupal 5. If someone wishes to take over as the 5.x maintainer I would consider it, but these days everyone should really upgrade to Drupal 6.x.