-
Notifications
You must be signed in to change notification settings - Fork 6
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
Dispatch some events #31
Comments
Hi @dextervip , this is a good idea... In some days I will update the bundle to make it better, adding some new functionalities and generally improving the code. I'd appreciate a lot if you could expand this suggestion with some use cases you like to get covered, so I can implement a better event system... |
Well my case use would be to listen events when emails are hard bounced and update my customer personal info entity to flag this email as invalid so my system stops sending and my agents know this is an invalid contact. |
Ok, I understand your point... I'm deeply refactoring the bundle and adding new features: this will be added in the version 2.0. I hope in a couple of weeks to finish... |
Tracked by #64 to make GitHub able to calculate the progress. |
Awesome! At moment, as a workaround I had to override the entire handle class (all methods are private so i couldn't override the specific method I needed, idk maybe refactor to protected would be good also). |
@dextervip , I've completely refactored the handlers: check the new code that should be more clear to understand. Start from the Any feedback about the new code is welcome! |
Hi, I would expect to process the bounces notifications in real time and, for that, having a controller somehow listening to these events. I can't find examples of this. |
@drzraf, the events are not fired in this moment. I deeply refactored the bundle and introduced a completely revamped command to configure the app with AWS SES. I plan to add the events in the next minor version. Any help on this, anyway, will be welcome! The main issue for this is this #64 I plan to add at least 5 events. More info in the linked issue. |
PS The documentation is not still updated: I'm updating it... |
Hey,
It would be nice to dispatch some events about bounces, complaints so that we can subscribe to make extra treatments in our app.
Regards
The text was updated successfully, but these errors were encountered: