For certain sites (such as security.drupal.org), administrators may want to get email notifications of *all* project issues and followups. It can be annoying to have to go into each project and set the email address for all issues of that project, because if a new project is created yet one forgets to set the email address for all issues of that project to be sent to, then the admin won't get notifications of all issues.

The attached patch adds a master subscription setting at /admin/project/project-issue-settings that allows an admin to give one or more addresses where all issues should go. The length of the text field is 100 characters long, so this should point to some kind of mailing list if there is the need for more than a few addresses here.

Files: 

Comments

hunmonk’s picture

Status: Needs review » Closed (won't fix)

i'm not sure it's a good time to commit something like this -- it feels like a hack on top of our ailing subscription system. the reason this would even be needed is that the form at project/issues/subscribe-mail (which already has options for all projects) doesn't work correctly.

my feeling is that we should put our energy into a proper redesign of that code, or start using subscriptions module.

aclight’s picture

I agree that it would be great to start using the subscriptions module, but that's not even finished yet. We've got a lot on our plate as far as using existing modules, and from what I understand this wouldn't be first on the list anyway.

I don't really see this as a hack anyway. As I understand, even if the subscriptions code were working properly it wouldn't be possible to do this anyway, because the mail would go to the email address associated with the user's account. So at least in the use case where the master subscription list would be a mailing list, someone would need to create a dummy account for the sole purpose of subscribing to all of the projects. That seems like a lot more of a hack than this setting does.

I personally don't have much of a need for this, but after seeing that this is a problem on s.d.o, and realizing it's a simple fix, I figured I'd roll a patch to add the functionality.

BTW, I haven't actually looked at the new subscriptions module yet, but I wonder if it would even offer this feature. I suspect that it's going to want to send to the user's email address as well, so we might be in the same boat as we would be in if we fixed the subscriptions code in project_issues now.

hunmonk’s picture

Status: Closed (won't fix) » Postponed

So at least in the use case where the master subscription list would be a mailing list, someone would need to create a dummy account for the sole purpose of subscribing to all of the projects. That seems like a lot more of a hack than this setting does.

i can see the limitation you're pointing out, and why this patch would solve it. in a larger context, i'm seeing a couple of things here:

  1. the current subscription code sux.
  2. there's been a lot of talk about using subscriptions module to replace it, but that's not a lock.

given those, it seems like a bad time to be introducing a new feature, especially since it adds an admin setting that could very likely disappear during the same major release because of redesign. IMO, adding this now is putting the cart before the horse, and we've had enough of that to contend with in the project module. i really think we should get some higher level decisions made before something like this enters the picture.

as such, this really should be postponed, not won't fix.

philbar’s picture

greggles’s picture

Status: Postponed » Active

No longer blocked now that flag is in.

dww’s picture

Version: 5.x-2.x-dev » 6.x-1.x-dev
Assigned: aclight » Unassigned
Status: Active » Postponed (maintainer needs more info)

Not blocked, but do we really still want this? If so, why? ;)

greggles’s picture

Well, we solved the immediate need for s.d.o by hardcoding each project. So, I don't really care about this any more but found it when closing old issues. I'm fine won't fixing it if it doesn't seem valuable to you.