notifyall in test mode #369

Merged

tobiasd merged 3 commits from 20160906-notify into master

2016-09-09 11:53:06 +02:00

tobiasd commented

2016-09-06 15:05:49 +02:00

(Migrated from github.com)

for friendica/friendica#2764

in test mode send the notification mail only to the admin(s) of the node
use the system set sender email or noreply

this also might need reflection in friendica/friendica#2768 so this needs to be decided before the other PR is merged.

for friendica/friendica#2764 - in test mode send the notification mail only to the admin(s) of the node - use the system set sender email or `noreply` this also might need reflection in friendica/friendica#2768 so this needs to be decided before the other PR is merged.

AlfredSK commented

2016-09-07 19:29:36 +02:00

(Migrated from github.com)

Okay. I copied the new code manually into my notifyall.php and did some tests. If email is set in site settings the mail is delivered from that address to the admin's email account. If email isn't set the mail is delivered from noreply... to the admin's email account. So, I think it is working like expected.

tobiasd commented

2016-09-09 11:06:30 +02:00

(Migrated from github.com)

I've added a DISTINCT to the SQL query so that if one email address is used for more then one account the notification mail is only send once to that address.

I've added a `DISTINCT` to the SQL query so that if one email address is used for more then one account the notification mail is only send once to that address.