-
-
Notifications
You must be signed in to change notification settings - Fork 200
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
Added email notification for committee subscribers #215
Conversation
|
||
foreach ($followers as $follower) { | ||
/** @var Adherent $follower */ | ||
$this->mailJet->sendMessage(CommitteeFeedNotificationMessage::create($follower, $message)); |
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.
You must send only one message with multiple recipients. See how I've done it with the events.
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.
Fixed in 28ab47a.
7afe4bb
to
28ab47a
Compare
28ab47a
to
5fdd94e
Compare
use Doctrine\Common\Persistence\ObjectManager; | ||
|
||
class CommitteeFeedHandler | ||
{ | ||
private $manager; | ||
private $committeeManager; | ||
private $mailJet; |
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.
mailjet lowercase?
…Dude) This PR was merged into the master branch. Discussion ---------- Added email notification for committee subscribers Closes [EM-48](http://jira.sensiolabs.net/browse/EM-48). Commits ------- 5fdd94e Added email notification for committee subscribers
Closes EM-48.