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

Use the DI container for creating transport factories #67

Merged
merged 8 commits into from
Jan 29, 2024

Conversation

mtangoo
Copy link
Contributor

@mtangoo mtangoo commented Jan 29, 2024

Q A
Is bugfix? ✔️/❌
New feature? ✔️/❌
Breaks BC? ✔️/❌
Fixed issues

Copy link

what-the-diff bot commented Jan 29, 2024

PR Summary

  • Updated CHANGELOG for new version
    The CHANGELOG has been updated for the new version (4.0.0). This comes with several enhancements such as using Dependency Injection (DI) container for creating factories, incorporating 'logger proxy' as a dependency, and ending support for outdated PHP versions 7.4 and 8.0.

  • Enhanced README with DI container usage
    The README documentation has been updated to include details about how to use the Dependency Injection (DI) container.

  • Upgraded dependencies in composer.json
    The project's dependencies on 'symfony/mailer' and 'symfony/mime' have been upgraded to version 7.0.

  • Refactor of Mailer class in source code
    The 'Mailer' class has been refactored for more efficient operations. There are several updates, including the use of 'DynamicLogger' (when available), enhancing the 'getTransportFactory()' method, and the inclusion of a new 'sendMessage()' method.

  • New DI Container test
    A new test to manage the Dependency Injection container usage has been added to our suite of validation tests.

@@ -1,11 +1,15 @@
Yii Framework 2 Symfony mailer extension Change Log
================================================

4.0.0 Jan 29, 2024
------------------
- Enh: Use DI container for creating factories (sammousa)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should go into 4.0.0 ↓

@mtangoo mtangoo merged commit 356488e into yiisoft:master Jan 29, 2024
0 of 7 checks passed
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