-
Notifications
You must be signed in to change notification settings - Fork 48
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
"hide_filter" property in withParams method on embed builder only hides one filter #159
Comments
@nuwen - Did you ever figure this one out? |
Hi @bryans99 , Can you please look into this issue and help with some workaround or solution? Is there any way to prioritise this bug? It would be great help. |
Hello, this would be really helpful indeed. And it should be doable as it's possible when embedding without the SDK. Do you think I could propose a PR @bryans99? |
Hello, I opened PR #177 to fix it, I hope it will suit |
PR #177 (released in @looker/embed-sdk@1.8.6) fixes the signature of
but even with this change looker only hide one filter: the last one in the array. Given how the parameters are stringified here embed-sdk/src/embed_builder.ts Line 50 in 23e6ab9
It seems to me the server can't handle querystring with repeated parameters |
@nuwen did you get a chance to test the new release of As @brunouber mentioned, this is still not usable cause the Looker back end does not handle the multiple parameters at all - or at least so it seems... Wonder if anybody has a workaround for this or who to ping to get some feedback.. :-) |
Hello
I have have an embedded dashboard using
createDashboardWithId
that has many filters, but I need to hide a few specific filters from the dashboard.I am trying to hide the filters "Date selector" and "Date filter", code example below.
I have tried
Ends up only hiding "Date filter", because it does not allow repeat properties
Does not work, results in showing both filters
Obviously doesn't work because it expects a string.
Thanks for looking!
Link to
hide_filter
docs https://cloud.google.com/looker/docs/filters-user-defined-dashboards#hiding_dashboard_filtersThe text was updated successfully, but these errors were encountered: