Feature Request: Notifications in GetReportsFromMailboxCommand

[ThomasTr]

Feature Request:
Notifications (via mail?) in GetReportsFromMailboxCommand for entries containing errors. This would allow the system to run largely autonomously and errors would be detected automatically without having to constantly check in.

If you accept pull requests, I could imagine helping out here.

[antedebaas]

I’m all for it!
This was already in my mind. Just didn’t have the time.
A pull request would therefore be appreciated.

I implemented the use of pushover to send notifications and would prefer that solution over email personally.
But maybe it can be tied into the notifications code to also send email and have it all implemented in one layer. (Eg send notification and/or email depending on configuration)

There is already a switch to send notifications which can be disabled via config.
I was thinking of extending that to include an ‘only or error’ option.

On the long term I want configuration to be user bound but tbh the who multiuser part needs to be thought trough so I’m thinking of leaving it in the generic config for now.

What do you think?

[ThomasTr]

Hi,

ok, will dig a little deeper in your code and come back.

[ThomasTr]

Hi Ante,

for now, I will stuck with the pushover notifications and have extended them so that they are only sent if the status is not good.

Here is my PR: #212