-
Notifications
You must be signed in to change notification settings - Fork 121
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add customizable adhoc SQL query for select adhoc values #655
Conversation
This is just not works
|
Signed-off-by: Eugene Klimov <[email protected]>
Hi 654 was merged, any update on this fix? |
but i run
|
@antip00 instead of create tests for already tested macros, better will create test which cover works with adhoc macros and cover new functionality |
@lunaticusgreen still get error and remove NL (press ) |
src/datasource/datasource.ts
Outdated
@@ -413,7 +416,7 @@ export | |||
createQuery(options: any, target: any) { | |||
const queryModel = new SqlQuery(target, this.templateSrv, options); | |||
// @ts-ignore | |||
const adhocFilters = this.templateSrv?.getAdhocFilters(this.adHocFilter?.datasource?.name || 'clickhouse') || [] | |||
const adhocFilters = getAdhocFilters(this.adHocFilter?.datasource?.name || 'clickhouse',this.uid ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong fix
we don't have default data source name
remove clickhouse
here
@lunaticusgreen e2e test failed just add new dashboard -> add new visualization (default clickhouse-direct datasource) got following error
|
If you change adhoc variable in panel, it do not changes in reformatted query |
@antip00 Hey, I think it's a bug with scenes, I've created bugreport, let's wait for grafanas response |
We probably will disable scenes for some time |
@lunaticusgreen http://localhost:3001/d/Bnh59Trnk/24-adhoc-2b-24-variable3a-text-formatting?editPanel=2 all works fine changing in adhoc control changed in Generated SQL looks like we broke it in 2.3.4 the same URL issue-330 branch little bit different behavior
panel in 3000 port
this is definitely bug and shall be fixed |
Improving adhoc tests. [WIP]
fix #330 , can be merged after #654