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:
- 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 reminderEnabling 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
Comment | File | Size | Author |
---|---|---|---|
per-project-issue-email-settings-d6.png | 61.86 KB | dww |
Comments
Comment #1
dwwBased 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...
Comment #2
dwwRe: B... this is somewhat interesting:
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...
Comment #3
drummLet'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.
Comment #4
dwwSome more stats:
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.
Comment #5
dwwhttp://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
Comment #6
dwwp.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.
Comment #7.0
(not verified) CreditAttribution: commenteduse 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