Author Topic: checkbox for Groups to say 'can use for Test Mail' (Read 1440 times)

petednz · March 15, 2011, 10:04:13 pm

to avoid human error that can result in Test Mailing going to a Group who shouldn't have been selected, could we have a checkbox for Groups to say 'can use for Test Mail' - and therefore only see such 'can use for test group' in the selector in CiviMail Step 4?

Dave Greenberg · March 16, 2011, 10:34:08 am

Peter - Sounds like a good idea. Would be fairly simple to implement since the other "Group Type" options are stored in option_value table. So I think steps would be something like this:

1. Add new option_value row for "Mailing Test List"
2. Might need to fix New / Edit group form - but I think it might just pick up the new option value as another checkbox
3. Modify form code for "Test Mailing" form to only expose the right group type. (NOTE: I think the Mailing Test List groups would be excluded from the "Select Recipients" form - since they are not "Mailing Lists". This is probably desired behavior ??)
4. For upgrade, insert new option_value row.

I'll ping Lobo for another opinion but I think a patch which does the above would be welcome.

Donald Lobo · March 16, 2011, 10:59:45 am

so i'm a bit conflicted about this.

i think it introduces another layer which adds another barrier for some people (and improves the process for some other people)

i do think that changes like this should be done via the hook system

lobo

petednz · March 16, 2011, 11:16:35 am

well i think some default barriers that avoid human error, such as sending the Test Mailing to the entire database, is along the lines of giving people 'Trash' so inadvertent deletes can be undone.
Happy to see if we can supply the patch - but if our persuasive powers fail then so be it.

petednz · March 16, 2011, 11:22:13 am

And would probably go the route
Group Type =
CiviMailing
CiviMail Testing
Access

CiviTeacher.com · March 16, 2011, 11:23:40 am

I actually can't believe this hasn't yet happened to one of my clients before! When you think that you're only missed click away from accidentally sending 14,000 people "testing 1-2-3" it is a definite upgrade that is necessary.

Perhaps as an easier solution would just be to have a popup that says:

"Are you SURE you want to send this test mailing to "The Whole Lot" Group: 14,000 people?"

xavier · March 16, 2011, 11:29:16 am

Hi,

Pete, don't put checkboxes when some combination aren't meaningfull (eg.
mailinglist uncheck
test mailing list checked)

I think replacing the mailing list checkbox with a select/radio

- not for mailing
- mailing list
- test & mailing list

Dave I do send mailing to "test mailing lists" (eg. staff).

X+

P.S. As an aside, I'm still not convinced by adding "Test Mailing" in the object and the sender is a good idea. For me, a test should be as close as possible from the real sending, ie. with the real name and real subject, not messing them around. Adding something discreet in the footer seems to be a better option IMO.

Dave Greenberg · March 17, 2011, 10:06:09 am

Xavier - I agree regarding "Test Mailing" in subject and name.

CiviCRM Community Forums (archive)

News:

Author Topic: checkbox for Groups to say 'can use for Test Mail' (Read 1440 times)

petednz

checkbox for Groups to say 'can use for Test Mail'

Dave Greenberg

Re: checkbox for Groups to say 'can use for Test Mail'

Donald Lobo

Re: checkbox for Groups to say 'can use for Test Mail'

petednz

Re: checkbox for Groups to say 'can use for Test Mail'

petednz

Re: checkbox for Groups to say 'can use for Test Mail'

CiviTeacher.com

Re: checkbox for Groups to say 'can use for Test Mail'

xavier

Re: checkbox for Groups to say 'can use for Test Mail'

Dave Greenberg

Re: checkbox for Groups to say 'can use for Test Mail'