Skip to content
This repository was archived by the owner on Jan 23, 2025. It is now read-only.

Conversation

@lha-wd
Copy link

@lha-wd lha-wd commented Mar 29, 2019

Hello everybody.

First of all great job on the panel.

I was missing the functionality of having a link for each of the generated circles based on the datasource-entries.

A textbox is now added, which can be used to specify the columnname (on table data) containing the link.

The link is then used to add an onclick listener to the circle.
I used code-fragments from PR #129 for this.

If the link is empty for some reason (for example: no link in datasource) the onclick listener will not be created.

I was only able to test it properly with MSSQL so any hints at problems with other datasources are welcomed.

With best regards,
Leo

rouba002 and others added 24 commits November 7, 2017 18:10
Make mouse wheel zoom optional
upgrade to latest stable leaflet version: 1.3.1
Use latitude and longitude if given in table data
Adding Elasticsearch in supported databases
Adds an intro text and resizes some of the images.
This allows a user to map these fields for those datasources
that do not allow aliases for fields.
Fixes the grafana#150 PR so that the tests pass and the fix works.
which occurs more frequently in Grafana 6.0
@CLAassistant
Copy link

CLAassistant commented Mar 29, 2019

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.
3 out of 4 committers have signed the CLA.

✅ rouba002
✅ leonhardhaas
✅ clompsy
❌ daniellee
You have signed the CLA already but the status is still pending? Let us recheck it.

@lha-wd lha-wd closed this Apr 1, 2019
@amotl
Copy link

amotl commented Apr 27, 2019

For all finding this: Looks like this PR has been superseded by #190 right away.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants