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
Usage of E-Mail Managers within CommandHandlers #15794
Usage of E-Mail Managers within CommandHandlers #15794
Conversation
Rafikooo
commented
Feb 1, 2024
Q | A |
---|---|
Branch? | 1.13 |
Bug fix? | no |
New feature? | no |
BC breaks? | no |
Deprecations? | no |
Related tickets | #15774 |
License | MIT |
c9ea89e
to
4d1ed66
Compare
Bunnyshell Preview Environment deletedAvailable commands:
|
4d1ed66
to
de746d6
Compare
de746d6
to
e419871
Compare
d0f9604
to
270d74b
Compare
e4cbf85
to
c3e0d57
Compare
src/Sylius/Bundle/ApiBundle/CommandHandler/SendContactRequestHandler.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/CoreBundle/Mailer/ResetPasswordEmailManager.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/CoreBundle/Resources/config/app/sylius/sylius_mailer.yml
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/CoreBundle/Resources/views/Email/accountVerification.html.twig
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/CoreBundle/Tests/Mailer/AccountRegistrationEmailManagerTest.php
Outdated
Show resolved
Hide resolved
bb1ff3c
to
e2569de
Compare
...ius/Bundle/ApiBundle/spec/CommandHandler/Account/SendAccountRegistrationEmailHandlerSpec.php
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/CoreBundle/Tests/Mailer/AccountVerificationEmailManagerTest.php
Outdated
Show resolved
Hide resolved
…esetPasswordEmailManager
…ationEmailHandler
…itory arguments within command handler classes
793df91
to
ef2b428
Compare
src/Sylius/Bundle/ApiBundle/CommandHandler/Checkout/SendOrderConfirmationHandler.php
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/CommandHandler/SendContactRequestHandler.php
Show resolved
Hide resolved
src/Sylius/Bundle/ApiBundle/CommandHandler/SendContactRequestHandler.php
Show resolved
Hide resolved
src/Sylius/Bundle/CoreBundle/Resources/config/app/sylius/sylius_mailer.yml
Show resolved
Hide resolved
src/Sylius/Bundle/CoreBundle/Resources/config/services/mailer.xml
Outdated
Show resolved
Hide resolved
src/Sylius/Bundle/CoreBundle/Resources/views/Email/accountVerification.html.twig
Show resolved
Hide resolved
<service id="sylius.mailer.account_registration_email_manager" | ||
class="Sylius\Bundle\CoreBundle\Mailer\AccountRegistrationEmailManager" | ||
> | ||
<argument type="service" id="sylius.email_sender" /> | ||
</service> | ||
<service id="Sylius\Bundle\CoreBundle\Mailer\AccountRegistrationEmailManagerInterface" alias="sylius.mailer.account_registration_email_manager" /> | ||
|
||
<service id="sylius.mailer.account_verification_email_manager" | ||
class="Sylius\Bundle\CoreBundle\Mailer\AccountVerificationEmailManager" | ||
> | ||
<argument type="service" id="sylius.email_sender" /> | ||
</service> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's minor but if you don't forget – inline id
with class
in a separate PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc: @Wojdylak
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, I didn't check the others :|
Thank you, @Rafikooo! |