-
Notifications
You must be signed in to change notification settings - Fork 154
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
Extract Swift mailer capability to new package #131
Comments
I like the idea, because it will allow to use custom mailing script if I understood it correctly? So for example I can call my own _mail function? |
Yes exactly |
Hi @fezfez, Would you be able to submit a pr for this? |
why you don't use phpmailer? |
@fezfez Wondering if you did any work on this? I'm seeing messages that swiftmailer is abandoned and should be replaced and just started to look into it. They appear to be recommending Symfony Mailer as the replacement. Regardless, I'd like to help if I can. |
helllo @jsokol, see https://github.com/job-runner/job-runner |
Hi,
I would like to drop the
swiftmailer/swiftmailer
dependencies from this package.To achieve this i propose to add a new interface JobbyNotifier such has :
and new repo such has
jobbyphp/jobby-swiftmailer
.the
jobbyphp/jobby
package could suggest to installjobbyphp/jobby-swiftmailer
for mails capability.A good alternative will be to add
symfony/notifier
but this will drop php 5.6, 7.0, 7.1.How do you feel about this proposal ?
Thanks
The text was updated successfully, but these errors were encountered: