Converted alerts table to use bootgrid #693

Merged
merged 3 commits into from Mar 31, 2015

Projects

None yet

2 participants

@laf
Member
laf commented Mar 30, 2015

The only thing missing at the moment is the colored rows as I've not worked out how to do this in bootgrid yet :(

@laf
Member
laf commented Mar 31, 2015

Rather than color each row (which appears to be difficult in bootgrid) I've taken to add a status column to indicate the alerts status:

http://pasteboard.co/2dKd5v8D.png

If people are happy with this then it's good to go.

@f0o
Member
f0o commented Mar 31, 2015

Good for me. Shall I merge?

@f0o f0o self-assigned this Mar 31, 2015
@laf
Member
laf commented Mar 31, 2015

I'm happy if others are.....no idea what's gone on with scrut on this one. Pretty sure it's not accurate.

@f0o
Member
f0o commented Mar 31, 2015

Yeah no worries about scrut. It seemsbto have forgotten about all the old issues heh

@f0o f0o merged commit 9cb859f into librenms:master Mar 31, 2015

1 check passed

Scrutinizer 2295 Issues, 19 Patches
Details
@laf laf deleted the unknown repository branch Mar 31, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment