Splitting this off from #1560012: Port per-user issue notification email functionality to D7 since it's more involved and we need to decide what we're doing first.

In D6, we have a bunch of custom settings on project nodes for some per-project email notification functionality. This is the "Issue e-mail options" fieldset on the "Issues" subtab when editing an issue:

D6 per-project email notification settings

A) Weekly critical issues report
"To get a weekly digest of critical issues specify an e-mail address."
B) Issues e-mail address
"If you wish to receive a copy of all the issues to a central location specify an address here. Note: the copy will contain links to file attachments." Then there's a wall of checkbox filters for categories and status values.
C) Monthly reminder
Enabling this will send a monthly reminder to users that have open issues registered.

A is kind of weird, but apparently used:

mysql> SELECT COUNT(nid) FROM project_issue_projects WHERE mail_digest != '';
+------------+
| count(nid) |
+------------+
|        854 | 
+------------+
1 row in set (0.01 sec)

Among the 854 is the Drupal.org infrastructure queue. :/

B is sort of unnecessary now that we've got the good per-user settings, but again, still used:

mysql> SELECT COUNT(nid) FROM project_issue_projects WHERE mail_copy != '';
+------------+
| COUNT(nid) |
+------------+
|       1784 | 
+------------+
1 row in set (0.02 sec)

In some ways, it's nice to be able to subscribe an address that isn't necessarily associated with a user account, but it's yucky to have to implement it separately. It'd also be nice to be able to use those kinds of filters for the primary follow-flag-based notifications (see #963704: Add ability to subscribe by component) but that's out of scope for the d.o D7 port.

C is totally weird and is now gone from the 7.x-2.x branch entirely. See #700616-6: Add a way for users to opt-in for reminder emails about open issues per-project

Before we do anything, I'd love to hear what drumm thinks, so I'm going to assign this to him for feedback.

Thanks,
-Derek

CommentFileSizeAuthor
per-project-issue-email-settings-d6.png61.86 KBdww
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dww’s picture

Based on an IRC chat with tvn, we agreed C should die (for now):
#700616-6: Add a way for users to opt-in for reminder emails about open issues per-project

Still trying to figure out what to do about A and B...

dww’s picture

Re: B... this is somewhat interesting:

mysql> SELECT count(pip.nid) FROM project_issue_projects pip INNER JOIN multiple_email m ON pip.mail_copy = m.email WHERE pip.mail_copy != '';
+----------------+
| count(pip.nid) |
+----------------+
|           1294 | 
+----------------+
1 row in set (0.05 sec)

So, most of the 1784 total users of this feature are just pointing to an email address that's already someone's actual email address (and in 1098 of the cases, it's the email in {users}.mail -- the primary email for the account). There are only 491 cases where this points elsewhere.

I obviously can't paste those email addresses here, but it's an interesting mix. In a lot of cases, it's clearly some Drupal shop's "development" or "devteam" mailing list/alias. In some cases, it's just some random gmail account that's neither in {users}.mail nor {multiple_email}.email (e.g. another account someone (presumably) controls that they didn't bother to register as an official address for their account). In 45 cases, it's a user+something@gmail account...

Anyway, I suspect a lot of this could be solved if we had #1966218: Infrastructure to support organization accounts and let organization users subscribe to issues, too. I know the user+something@gmail crowd likes doing it that way, but they could almost certainly achieve the same filtering based on the subjects of these emails, so I'm not too worried about that. The random other address case could either be ignored (tell them to use primary) or solved via letting users control which of their multiple emails they wanted their issue notifications to.

So, I lean towards killing B with the understanding that #963704: Add ability to subscribe by component (or filters more generally) should be solved Soon(tm).

Just wanted to record all this data here for now. Still not actually making a decision...

drumm’s picture

Let's kill all of these. If we find they were wanted after launch, volunteers can look at the the best way to build these in a less-custom way using other modules.

dww’s picture

Assigned: drumm » dww

Some more stats:

mysql> SELECT COUNT(nid) FROM project_issue_projects WHERE mail_copy_filter_state LIKE '%1%';
+------------+
| COUNT(nid) |
+------------+
|       3327 | 
+------------+
1 row in set (0.03 sec)

mysql> SELECT COUNT(nid) FROM project_issue_projects WHERE mail_copy_filter LIKE '%1%';
+------------+
| COUNT(nid) |
+------------+
|          0 | 
+------------+
1 row in set (0.03 sec)

So, a lot of projects filter by state, but not a single one filters by category. ;)

Anyway, I'm fine with killing all of these for now in the interest of getting the port done. However, I highly doubt there are generic solutions for these features in other modules, short of something like Rules (which we already ruled out).

Assigning back to myself to actually clean up the remaining references to these features in D7 -- stay tuned.

dww’s picture

Title: Port per-project email notification functionality to D7 » Remove per-project email notification functionality for D7
Status: Active » Fixed

http://drupalcode.org/project/project_issue.git/commit/3e4eaa9 removes the last traces of A and B from the 7.x-2.x branch. If/when we want any of this functionality, we can re-add it in a sane way.

Thanks,
-Derek

dww’s picture

p.s. tvn and I just started a Google doc for the d.o D7 upgrade FAQ so we can record stuff that end-users might need to know after we launch. I added an entry for this, so we'll remember to notify people that they need to subscribe themselves if they want this functionality to continue.

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

Anonymous’s picture

Issue summary: View changes

use letters not numbers to refer to the 3 parts of this to avoid confusion with comment numbers, and C (reminder emails) is now resolved/handled at #700616 --dww