Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Mailer] document FileTransport #11702

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

Closed
wants to merge 2 commits into from
Closed

[Mailer] document FileTransport #11702

wants to merge 2 commits into from

Conversation

Kocal
Copy link
Member

@Kocal Kocal commented Jun 8, 2019

Hi, this time it's not a Encore PR πŸ˜„

It needs symfony/symfony#31947.

Thanks!

@noniagriconomie
Copy link
Contributor

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

@wouterj wouterj added the Waiting Code Merge Docs for features pending to be merged label Jun 10, 2019
@Kocal
Copy link
Member Author

Kocal 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
Copy link
Contributor

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
Copy link
Member Author

Kocal commented Jun 12, 2019

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

@OskarStark
Copy link
Contributor

The code-PR is closed. Can we close here @Kocal ?

@Kocal
Copy link
Member Author

Kocal commented Aug 14, 2019

Ah yes we can close it, I forgot about it.

@Kocal Kocal closed this Aug 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Reviewed Waiting Code Merge Docs for features pending to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants