Healthcheck notifications

[tmasternak]

This PR introduces health check notification capabilities to ServicePulse. Notifications can be delivered as emails.

[SzymonPobiega]

I can't really read or review JS. I can try running it and clicking here and there. Would that be ok @tmasternak ?

[tmasternak]

@SzymonPobiega sounds good.

[SzymonPobiega]

Looks good. The only thing that looks a bit odd is this:

but I am not sure we can do anything with it. Certainly not with my CSS/HTML skills 😂

[mikeminutillo]

I'm not terribly confident reviewing js but it looks good to me.

It might make sense to have the test connection button on the edit. That requires a bit of a re-architectue of the back-end, and it's probably not worth it for this first release.

Does ServicePulse have a minim required ServiceControl version, because this will increase it. What happens in this UI with an older version of ServiceControl?

[tmasternak]

It might make sense to have the test connection button on the edit. That requires a bit of a re-architectue of the back-end, and it's probably not worth it for this first release.

I had a chat with Sergio about that and we decided that the current location is good enough

Does ServicePulse have a minim required ServiceControl version because this will increase it. What happens in this UI with an older version of ServiceControl?

It looks that this is the current minimum which looks quite outdated. I tried to spike an approach in which the tab hides when the ServiceControl version is less than 4.17 (we already have code to do version parsing etc.). That being said there seems to be more re-work needed around Angular controllers for the configuration section. I propose to create a separate issue for that.