-
Notifications
You must be signed in to change notification settings - Fork 1
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
Email Library #13
Comments
I'd love this to happen. We probably need to improve our CC/BBC support, but I think it's a |
Of course. |
So is this an easy or a complicated task? |
I have no idea, because I had no time to play with Laravel. |
This has nothing to do with Laravel specifically. |
I wouldn't have thought so, it's just a matter of ensuring we follow a standard of coding, and keep the separation of dependencies correct. |
It's most there, it's just creating a standalone set of classes that could be used by any code, without being Symphony specific. I think the main tie we have is the Configuration class, and that's a pretty loose dependency. |
Yeah, we just need to specify that it needs configuration injecting into it, and show an example of that configuration. |
Quite complicated. In fact, it will require almost an entire rewrite. For Symphony the code works fine, because Symphony does not require its code to be automatically testable. So the email code wasn't written with that in mind. |
Wasn't the purpose of using a framework not having to do stuff like this ourselves and instead just using the framework for it? On a side note, open source webmailer Roundcube recently decoupled its core functions for IMAP access and SMTP email sending into Roundcube framework. |
One thing Symphony can be proud of is its simple, small and reliable email system. In the light of Next, wouldn't it be a good idea to release it as a standalone library? Might even be an interesting component for Laravel (which is currently relying on the huge dinosaur Swiftmailer).
The text was updated successfully, but these errors were encountered: