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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Mailer] document FileTransport #11702

Open
wants to merge 2 commits into
base: 4.4
from

Conversation

Projects
None yet
4 participants
@Kocal
Copy link
Contributor

commented Jun 8, 2019

Hi, this time it's not a Encore PR 馃槃

It needs symfony/symfony#31947.

Thanks!

@noniagriconomie

This comment has been minimized.

Copy link
Contributor

commented Jun 8, 2019

Can also be usefull in dev env
Maybe add a more generic sentence for both env?

@Kocal

This comment has been minimized.

Copy link
Contributor Author

commented Jun 12, 2019

I don't really know why you would use it in dev env, but I may be wrong. 馃槬
What can be your use case for example?

@noniagriconomie

This comment has been minimized.

Copy link
Contributor

commented Jun 12, 2019

Because you may not have install any mailer soft
And want to check the rendered email

The idea is to not document as 芦聽only for test env聽禄 and let developers choose, of course test env is the best usage but other can be usefull also :)

@Kocal

This comment has been minimized.

Copy link
Contributor Author

commented Jun 12, 2019

Hum that's right, I've updated the documentation :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can鈥檛 perform that action at this time.