Closed eddiewebb closed 3 years ago
Quite sure this is change in Laravel 6.x to Laravel 8.x which changed mail handler. I'll see if I can play with a PR but I don't know much about laravel internals.
https://github.com/illuminate/mail/blob/8.x/MailServiceProvider.php#L26
I don't think this extension has been updated for beta 16 yet. But I am following because I'm also waiting for it before I can upgrade :)
Indeed, nobody from FoF has tested the extension on beta 16 yet, it's likely it will require an update.
We are (slowly) getting to the end of the list of extensions to update.
@clarkwinkelmann - yep, understanding it was likely a beta16 compatability, I wanted to contribute if I could.
name
to the Mailer we provider as Laravel 7.x added support for multiple mailers.
Bug Report
Current Behavior All emails are failing with a 500. Logs report the error below.
Steps to Reproduce
Expected Behavior Email is sent.
Screenshots
Environment
Possible solution(s) line 29 here throws the error, but not sure why https://github.com/FriendsOfFlarum/pretty-mail/blob/master/src/Providers/MailerProvider.php#L29
Additional Context LOgs