Added Alerting `dirty` #370

Merged
merged 6 commits into from Dec 21, 2014

Projects

None yet

3 participants

@f0o
Member
f0o commented Dec 17, 2014

Adds Alerting like in #366 but without cleaning up the old alerting-code, therefore dirty.

@f0o f0o referenced this pull request Dec 17, 2014
Closed

Added Alerting #366

@f0o
Member
f0o commented Dec 17, 2014

Related:
#369 API
#367 GUI

@laf
Member
laf commented Dec 19, 2014

Is it possible for ajax_rulesuggest,.php to still send back a table name even if device_id is empty?

@laf
Member
laf commented Dec 19, 2014

Also if possible, carry on honouring these two config options to force alerts to only be sent to this contact:

$config['alerts']['email']['default'] = 'neil@lathwood.co.uk'; // Default alert recipient
$config['alerts']['email']['default_only'] = TRUE; // Only use default recipient

f0o added some commits Dec 19, 2014
@f0o f0o Added:
	Arithmetic support in rules
	Braces support in rules
	Changed wildcard character to '@'
7ec2627
@f0o f0o Added:
	Honouring `$config['alerts']['email']['default']` and `$config['alerts']['email']['default_only']`
fd93042
@f0o f0o Added:
	Default Contact's Name (NOC)
6ac4776
@f0o
Member
f0o commented Dec 20, 2014

Anything else required prior merge? :)

@paulgear paulgear merged commit 6ac4776 into librenms:master Dec 21, 2014

1 check passed

Scrutinizer 10 new issues
Details
This was referenced Dec 23, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment