Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More complete fix for laf:issue-87 #96

Merged
merged 3 commits into from Feb 23, 2014
Merged

More complete fix for laf:issue-87 #96

merged 3 commits into from Feb 23, 2014

Conversation

paulgear
Copy link
Member

Anyone care to review before I merge? This works on my laptop and my home server, but they have limited client devices.

laf and others added 3 commits February 14, 2014 07:53
 - Align all columns
 - Move all of the common parts of each query to the front so differences can be seen easily
 - Fix mixup of deleted vs. ignored
 - Ensure deleted & ignored are checked correctly in each query
@laf
Copy link
Member

laf commented Feb 23, 2014

Looks good to me, you've certainly done a more thorough job than I did :/

paulgear added a commit that referenced this pull request Feb 23, 2014
More complete fix for laf:issue-87
@paulgear paulgear merged commit f5f580e into librenms:master Feb 23, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants