fix: Fix state_indexes for state overview sensors #5191

Merged
merged 1 commit into from Dec 18, 2016

Conversation

Projects
None yet
5 participants
@laf
Member

laf commented Dec 18, 2016

DO NOT DELETE THIS TEXT

Please note

Please read this information carefully. You can run ./scripts/pre-commit.php to check your code before submitting.

The only area I can see that we use columns from the joins in this query is state_index_id so including this in the output of the query.

@LibreNMS-CI

This comment has been minimized.

Show comment
Hide comment
@scrutinizer-notifier

This comment has been minimized.

Show comment
Hide comment
@scrutinizer-notifier

scrutinizer-notifier Dec 18, 2016

The inspection completed: No new issues

The inspection completed: No new issues

@E-t-z

This comment has been minimized.

Show comment
Hide comment
@E-t-z

E-t-z Dec 18, 2016

Contributor

Works for me.

Contributor

E-t-z commented Dec 18, 2016

Works for me.

@Rosiak Rosiak merged commit e7d0d1d into librenms:master Dec 18, 2016

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@laf laf deleted the laf:issue-5180-1 branch Dec 18, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment