Skip to content

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