search

anryko / grafana-influx-dashboard

Grafana InfluxDB scripted dashboard
MIT License
132 stars 44 forks source link

don't load all graphs when only displaying the hosts list #51

Closed nordewal closed 8 years ago

nordewal commented 8 years ago

Don't load all graphs when only displaying the hosts list

anryko commented 8 years ago

Heh... looks like I introduced a bug in the last version. Thanks a lot for pointing this out! However, I prefer to fix it by adding "return" statement in bogus "if" to finish function execution after callback was triggered.

nordewal commented 8 years ago

This is debatable ;-) I generally agree to have returns in if statement if it makes it more readable. But here I see it as more readable using an if/else. Anyways thanks for responding quickly!