Added user defined titles for alert templates #1807

Merged
merged 5 commits into from Aug 30, 2015

Projects

None yet

3 participants

@f0o
Member
f0o commented Aug 30, 2015

Fixes #1537

@f0o f0o Added user defined titles for alert templates
ac7d0c9
@f0o f0o added the Alerting label Aug 30, 2015
@f0o f0o self-assigned this Aug 30, 2015
f0o added some commits Aug 30, 2015
@f0o f0o Fix default template assignment
9057d48
@f0o f0o Added JavaScript to modal
68f9812
@f0o f0o removed their assignment Aug 30, 2015
@f0o
Member
f0o commented Aug 30, 2015

Good for merge @laf / @paulgear :)

@f0o
Member
f0o commented Aug 30, 2015

just FYI, the title does not allow placeholders just yet, I need a bit more restructuring to get placeholders into that.

@f0o f0o Add placehoders to custom titles
a833d46
@f0o
Member
f0o commented Aug 30, 2015

Placeholders now supported

@f0o f0o Fixed edit-javascript
bea0587
@paulgear paulgear merged commit 98d99a6 into librenms:master Aug 30, 2015

2 checks passed

Auto-Deploy Build finished.
Details
Scrutinizer 1 new issues
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment