Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can't use multiple emails when using netteMailer option set to TRUE #308

Closed
Gappa opened this issue Jun 28, 2018 · 0 comments
Closed

Can't use multiple emails when using netteMailer option set to TRUE #308

Gappa opened this issue Jun 28, 2018 · 0 comments

Comments

@Gappa
Copy link

Gappa commented Jun 28, 2018

Version: 2.5.0

Bug Description

When using the new default option of netteMailer: true, multiple emails can't be set in config.neon:

tracy:
	email:
		- email1@example.com
		- email2@example.com

This ends up throwing an error by itself:

Nette\Utils\AssertionException The header 'To' expects to be email, string given.

When trying to add the multiple emails:

.../tracy/tracy/src/Bridges/Nette/MailSender.php:47	source  Nette\Mail\Message->addTo
$email	"email1@example.com, email2@example.com"

Steps To Reproduce

Set Tracy email config as above and trigger an error in the application that will send an email.

Expected Behavior

Email message containing the errror is successfully created.

Note

Maybe I'm not setting the emails correctly? I tried also this, but that also didn't work:

tracy:
	email: 'email1@example.com, email2@example.com'

Or did it previously (i.e. with netteMailer: false) work just "by accident"?

@Gappa Gappa changed the title Can't use multiple emails when using netteMailer option se to TRUE Can't use multiple emails when using netteMailer option set to TRUE Jun 28, 2018
@dg dg closed this as completed in 7b5d4f6 Jun 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant