grafana / influxdb-08-datasource

Data source plugin for InfluxDB 0.8.x
Apache License 2.0
6 stars 11 forks source link

Bug correction to work with infludb 0.8x #8

Open bloublou2014 opened 7 years ago

bloublou2014 commented 7 years ago

Hi, after some research problem comes from :

configuration error : database, username & password are not take into account from config partial via ctrl.current.xxx . (Based on grafana plugins, using ctrl.current.jsonData.xxx do the trick.) URL should be http://xxxxxx/db/MyDatabase/yyyy as previous post mention. bs-typeahead does not work since references to function in query_ctrl are not in scope

Tested on Grafana 3.1.1 with Influxdb 0.8 with direct connection.

2016-10-20_08-45-14

tinco commented 7 years ago

I can confirm this pull request solves an issue for us.

jabbors commented 7 years ago

I can also confirm that it works for Grafana 4.

weiyin commented 5 years ago

This fixes the plugin for us too. Otherwise the plugin is completely broken.

CLAassistant commented 2 years ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: bloublou2014
:x: pioupioupiou
You have signed the CLA already but the status is still pending? Let us recheck it.