Updated parsing of alert rules to allow | #2917

Merged
merged 1 commit into from Feb 7, 2016

Projects

None yet

2 participants

@laf
Member
laf commented Feb 3, 2016

Shouldn't break anything, I have tested against a variety of existing rules with both AND and OR without issues.

Fix #2917

@laf laf Updated parsing of alert rules to allow |
350345b
@laf laf added Bug WebUI labels Feb 3, 2016
@f0o f0o merged commit a7f1a98 into librenms:master Feb 7, 2016

1 of 2 checks passed

Auto-Deploy Triggered
Details
Scrutinizer No new issues
Details
@f0o f0o closed this Feb 7, 2016
@laf laf deleted the laf:issue-2885 branch Jan 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment