Add Open/Click events

[malas]

NotificationProcessor.php should also process the open and click Notifications from AWS and dispatch corresponding symfony events.

Ping #64

[Aerendir]

Hi @malas , thank you for highlighting this!

I'm in the process of better understanding how the system works and how to implement it.

This is not as simple as reading the responses from Amazon, but requires to think also at the way the bundle handles the configuration of topics.

Without speaking about GDPR, that for sure is not something that the bundle should take into account. But, as the bundle will make a lot easier to configure the tracking features, a mention, also if very small, should be done.

Are you able to implement this feature?

[malas]

After examining your bundle we have decided to roll our own solution.

The main difference is that our solution will only emit a coresponding symfony event on SNS notification an then one can hook any logic he wants.

[Aerendir]

Ok, as you like.

Just to understand better, if you like to answer me: currently, the bundle doesn't emit any event: this is a planned feature that will be released very soon.

Once the bundle will emit the Symfony events, which will be the difference with your own solution?

[malas]

Our solution will not have any additional logic bundled within like saving smth to database. Our viewpoint is to do one thing and do it very well. In this case it would be to receive and parse sns rmail notifications.

…

On Mon, 24 Sep 2018, 20:47 Adamo Crespi, ***@***.***> wrote: Ok, as you like. Just to understand better, if you like to answer me: currently, the bundle doesn't emit any event: this is a planned feature that will be released very soon. Once the bundle will emit the Symfony events, which will be the difference with your own solution? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#84 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AALqvHsbGpTqbi1xa6lxV8yJFjfojuuxks5ueRq2gaJpZM4Wrm3a> .

[Aerendir]

Ok, I understand your point. Anyway, thank you for your comment :)