-
Notifications
You must be signed in to change notification settings - Fork 47
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
withFilters doesn't actually set initial filter values #81
Comments
On version |
Hi @joeyorlando and @hitpopdimestop - I believe this is working as intended. The Basically, unlike I'd be curious to better understand your use case. For example, what kind of URL are you trying to pass in as the mySignedUrl value? |
Also pinging anyone who gave this a thumbs up to possibly provide more context? @dnaport22 @fantua @charmingelle @rbob86 |
Hi @fabio-looker. If there is a limitation in using withfilters for createDashboardWithUrl then why there is nothing about it in docs? Also, there can be done next improvements:
|
Hello, Just wanted to mention that I'm using the const db = LookerEmbedSDK.createDashboardWithId(dashboardId)
.appendTo(el)
.withParams({
_theme: JSON.stringify({
show_filters_bar: false,
}),
})
.withNext() // Using Dashboard next
.withFilters(
{
'End Date Param': '2022/04/07',
}
)
.build()
.connect() Using version |
Chiming in here, we have:
And the initial filters are not being set. If we do Version 1.8.1 |
Thank you! |
I was poking around the source code a bit. It look like:
EmbedBuilder.withFilters
sets the passed in object to `this._paramsEmbedBuilder.build
is called, which returns anEmbedClient
object),EmbedClient.connect
internally callsthis.createIframe
(since I'm usingcreateDashboardWithUrl
), which sets up aChatty
instance. I feel like the_params
should be sent into theChatty
instance here but they are not?Using version
1.6.0
The text was updated successfully, but these errors were encountered: