Skip to content

Conversation

@zupolgec
Copy link
Contributor

@zupolgec zupolgec commented Apr 23, 2020

Fixes #1

@mpociot when I suggested using SwiftMailer plugins I still hadn't see the inner workings of the plugin. Since it's tied to the Mailable, it's difficult to implement as a SwiftMailer plugin and provides no advantages.

I think the fix in this PR solves the problem in the easiest way possible. I just split the Mailers and added the same mailer method that is found in the MailFake class.

@zupolgec zupolgec force-pushed the fix-laravel7-new-mailer branch from e82d3b4 to d584333 Compare April 23, 2020 22:48
@mpociot mpociot merged commit dffa167 into beyondcode:master Apr 24, 2020
@mpociot
Copy link
Member

mpociot commented Apr 24, 2020

Thank you!

@zupolgec zupolgec deleted the fix-laravel7-new-mailer branch April 24, 2020 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mailer not implementing Illuminate\Contracts\Mail\Factory

2 participants