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

Adapt to Swiftmailer 5 and Swiftmailer 6 version of class Swift_ConfigurableSpool #279

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

thirsch
Copy link
Collaborator

@thirsch thirsch commented Feb 7, 2023

@thePanz as promised, here is a possible solution of the swiftmailer issue. What do you think?

Btw. as the swiftmailer package is discontinued but needs a small adjustment for php 8.2: Should we clone the official repo in fos1 as well or try to adapt to symfony/mailer?

@connorhu
Copy link
Collaborator

connorhu commented Feb 9, 2023

+1 for symfony/mailer wrapper as a mailer plugin.

@thePanz
Copy link
Member

thePanz commented Feb 10, 2023

For reference: #240 (comment)

@thirsch
Copy link
Collaborator Author

thirsch commented Feb 11, 2023

Do you have any experience, how to use the spool? I've thought about creating a test for it. While browsing the test sources, I've come across https://github.com/FriendsOfSymfony1/symfony1/blob/master/test/unit/mailer/fixtures/TestSpool.class.php which is referencing Swift_Mime_Message as well.

@connorhu
Copy link
Collaborator

connorhu commented Feb 13, 2023

I am currently using Swfitmailer with spool under Symfony5. I can figure out how to set it up here.

@thePanz
Copy link
Member

thePanz commented Feb 13, 2023

we might need to run the tests on the CI with --prefer-lowest to test both versions of the SwiftMailer, wdyt?

@connorhu
Copy link
Collaborator

@thirsch File(system) spool example configuration:

factories.yml

  mailer:
    param:
      delivery_strategy: spool
      spool_class: Swift_FileSpool
      spool_arguments:
        - "spoolpath"

image

@connorhu
Copy link
Collaborator

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.

3 participants