Skip to content
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

[3.x] Preparing for Symfony 6.0 & Laravel 9.0 #131

Closed
joostdebruijn opened this issue Dec 7, 2021 · 5 comments
Closed

[3.x] Preparing for Symfony 6.0 & Laravel 9.0 #131

joostdebruijn opened this issue Dec 7, 2021 · 5 comments
Milestone

Comments

@joostdebruijn
Copy link
Contributor

Detailed description

The release of Laravel 9.0 is scheduled for January 2022 and it ships with Symfony 6.0. Swift Mailer is replaced by Symfony Mailer.

Relevant pages and PR's to have a look at:

Context

This project provide some features I like which are not present in the Symfony Postmark bridge, most notably sending mails via templates. It is not likely that the Symfony Postmark Bridge will implement this functionality in their project for obvious reasons, looking at a similar case for Mandrill (symfony/symfony#37612).

Possible implementation

The changes in Symfony 6.0 and Laravel 9.0 are not backwards compatible and require a serious rewrite of this project (for example run the tests in https://github.com/joostdebruijn/laravel-postmark/tree/3.x). It may be interesting to have a look at the Symfony Postmark Bridge (https://github.com/symfony/postmark-mailer) and extend it where necessary to achieve feature parity with the module at this moment. This reduces maintance on this module while keeping the extra functionality.

Of course I'm very happy to help, but before spending time on this I would like to know what the maintainer of this package thinks of the upcoming changes and the future of this project.

Your environment

Include as many relevant details about the environment you experienced the bug in and how to reproduce it.

  • Laravel Postmark Version: 3.0.0?
  • Laravel Version: >= 9.0.0
  • PHP Version: >= 8.0.0
@craigpaul
Copy link
Owner

Hey @joostdebruijn, I'm thinking that v3 of this project could be a complete rewrite that supports Laravel 9 + Symfony Mailer exclusively. I'll add this to the v3 milestone. I'm hoping to have time for this over the holidays / in the new year, but I'll gladly accept PRs for any of the v3 milestone items.

@craigpaul craigpaul added this to the v3.0 milestone Dec 7, 2021
@craigpaul craigpaul changed the title Preparing for Symfony 6.0 & Laravel 9.0 [3.x] Preparing for Symfony 6.0 & Laravel 9.0 Dec 7, 2021
@craigpaul
Copy link
Owner

I got a small start on v3 tonight. Ended up deciding on a full-on rewrite, so I'll be posting some updates within this (and other relevant issues) as I go. The symfony/mailer is a bit strange, but it's starting to make more sense the further I get into this 👍

@joostdebruijn
Copy link
Contributor Author

Nice @craigpaul! If there is something I can help with, please let me know.

@craigpaul
Copy link
Owner

Just tagged v3.0.0-RC1 which addresses the upgrade from swiftmailer/swiftmailer to symfony/mailer in Laravel 9.0. Please feel free to take it for a spin and send in any issues / PRs needed before a full release can take place 👍

@joostdebruijn
Copy link
Contributor Author

@craigpaul I gave it a spin in a separate testproject and the implementation is looking great. Nice work!

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

No branches or pull requests

2 participants