Query Variable with multi selection generates unparsable array

[KrisztianKaszas]

Hi,
using 0.1.7. when I add a Query Variable with multiselect and select more than one items, the array of strings being generated is unparsable:

For example if I have selected MetricA and MetricB than it produces following:
"["MetricA","MetricB"]"

It should produce the following instead:
"["MetricA","MetricB"]"

Otherwise Query Variable with multiselects are unusable, since unparsable.

Thank you and best regards,
Krisztian

[KrisztianKaszas]

Ah, I know why: currently the only way to insert a Grafana Variable in Additional JSON Data is using double quotes. Like "$QueryVariable". Hence if the query variable is being converted to values, the surrounding double quotes remain. As such any array is being converted to a string.

I think I'll have to handle it when reading the json body ... although it would be great, if I could insert a Grafana Variable without surrounding double Quotes.

[simPod]

This is a feature we reworked a lot. It has huge churn rate and I believe is not perfect so this bug might be definitely valid.

This is the last refactor that has been done to it that also added first tests on it: #62. Can you reproduce the bug via test case or ideally fix it?

[gjask]

I believe the problem is that currently data source is unable to correctly interpolate and parse this. Probably because there is some kind of json validation happening even before variable interpolation.

The reason why user may need this is exactly when variable should contain json structure (eg. list or even object) instead of primitive type (eg. integer, string, ...). Correct way to address this should be to postpone all json linting and parsing after variable interpolation.

I would like to help myself with solving the issue, but sadly I am completely at loss with JS. Maybe if you could point me out how to write tests for this, I would at least provide some. :(

OT: Additional JSON data field is too small and not resizeable.

[simPod]

There are some interpolation tests in datasource.jest.ts

BTW the field size has been increased in #126

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.