Issue 372: Email notification to multiple emails

Reported by Mathias Bavay, Jan 5, 2010

It could be useful to allow sending email notifications to more than 
one email address: if a comma separated list of emails could be 
given, then multiple users could receive the notification.

Another possibility could be to use a mailing lists support (see 
issue 371) and provide only one email address that would be the one 
from a mailing lists (so for example any issue notification could be 
sent to a list like dev.project@indefero.net)

Comment 1 by Loïc d'Anterroches, Jan 15, 2010

Thanks for your input. I prefer the idea to send to a mailing list 
and use it to dispatch afterwards. 

I will focus on the mailing list side of the work and check and 
update issue 371 accordingly.

Labels: Type:Enhancement ~~Type:Defect~~
Status: WontFix

Comment 2 by Pierre MARECHAL, Jan 29, 2010

I agree with the 1st proposal. I don't see any reason to limit the 
number of email addresses.

I attach a patch proposal.

Pierre

bug_372.patch - 5.81 kB - view

Comment 3 by Michaël Baudin, Jul 30, 2010

I do not understand the technical reason behind the 
"wontfix" status of this ticket.
Indeed, if the tool can manage one e-mail, it can manage several 
ones.
Creating mailing list would require to use another external tool, 
thus creating an unnecessary complexity.

I work in the Scilab Consortium - Digitéo and I am managing more 
than 20 different projects on Scilab's Forge :
http://forge.scilab.org/

For example : 
http://forge.scilab.org/index.php/p/distfun/
http://forge.scilab.org/index.php/p/assert/
http://forge.scilab.org/index.php/p/docintrodiscrprobas/
http://forge.scilab.org/index.php/p/sci-ipopt/
http://forge.scilab.org/index.php/p/stixbox/
etc...

For some projects I am the administrator (they may be other admins 
as well), for some projects I am developer.
Currently, I miss most of the tickets and project changes because of 
the fact that only one e-mail can be configured.
Therefore, it is completely impossible to keep up-to-date for these 
projects. Recently, I accidentaly were informed that a ticket had 
been opened, for which I knew the solution.

Comment 4 by Michaël Baudin, Aug 18, 2010

Is there something new for this ticket ?

Comment 5 by Sylvestre Ledru, Aug 24, 2010

I agree it would be nice to have this feature.

Comment 6 by Thomas Keller, Nov 25, 2010

I think we should consider this patch as well - the reason why I do 
not commit it right away is that I think a text area with one email 
address per line might be a better solution. This would require some 
UI changes though.

Status: New

Comment 7 by Sylvestre Ledru, Nov 25, 2010

Are you considering doing the UI changes ?

It would have been nice from Loic to provide some feedbacks on this 
when Pierre Marechal proposed the patch.

Comment 8 by Thomas Keller, Nov 25, 2010

I'd like to get feedback on my proposal from Loic as well before I 
start on anything. I'm assigning the issue to him now, this makes it 
popup on his TODO list...

Owner: loic

Comment 9 by Michaël Baudin, Nov 25, 2010

Excellent news.

Comment 10 by Loïc d'Anterroches, Nov 29, 2010

Ok, accepted to put a comma separated list of emails.

Status: Accepted
Owner: tommyd

Comment 11 by Sylvestre Ledru, Nov 29, 2010

Great. Thanks!

Comment 12 by Thomas Keller, Dec 6, 2010

This patch has been applied in commit 51c42a6.

Changes wrt the original implementation:
- source notifications are handled as well (this was previously 
forgotten)
- ',', not ';' is used as separator
- a separate form field is used to validate and clean the input 
(less dependent on a specific Pluf version)
- the grey box on the project configuration page received a hint 
about the possibility to enter multiple emails and the input fields 
got a little larger

Labels: Milestone:Release1.1
Status: Fixed

Indefero