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
[FormBundle] Moved FormMailer dependency to separate listener #2391
[FormBundle] Moved FormMailer dependency to separate listener #2391
Conversation
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.
Hi @devigner, your PR needs some changes
- your PR title should look like
[SomeBundle] Fixed some code
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.
Hi @devigner, your PR passed all our requirements.
Thank you for contributing!
The 'SymfonyInsight: kunstmaanlabs / KunstmaanBundlesCMS' failure is caused by code outside my PR. |
@@ -3,6 +3,7 @@ | |||
namespace Kunstmaan\FormBundle\Event; | |||
|
|||
use Kunstmaan\FormBundle\Entity\FormSubmission; | |||
use Kunstmaan\FormBundle\Helper\FormPageInterface; | |||
use Kunstmaan\NodeBundle\Entity\AbstractPage; |
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.
Can you switch out all of the AbstractPage typehints because this is an unused use statement now.
Moved sending the email to a listener. The listener can be removed when no email needs to be send after form submission. For example when the form submission is used to create a MailChimp user.