latitude-dev / latitude

Developer-first embedded analytics
https://latitude.so
GNU Lesser General Public License v3.0
673 stars 26 forks source link

fix: runquery with parameters #387

Closed geclos closed 3 weeks ago

geclos commented 3 weeks ago

Describe your changes

When we try to run a subquery with runQuery and its parent query has parameters, the db throws an error as we were passing parameters for the subquery that the db didn't know what to do with.

This commit fixes this issue.

https://github.com/latitude-dev/latitude/assets/1948929/5b2a2b3a-2ffb-4f4f-b880-63b56f5afdb7

Checklist before requesting a review

changeset-bot[bot] commented 3 weeks ago

🦋 Changeset detected

Latest commit: f449cf93d25b181f0d33472810ee38616e533133

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 14 packages | Name | Type | | ----------------------------------- | ----- | | @latitude-data/base-connector | Patch | | @latitude-data/server | Patch | | @latitude-data/athena-connector | Patch | | @latitude-data/bigquery-connector | Patch | | @latitude-data/clickhouse-connector | Patch | | @latitude-data/databricks-connector | Patch | | @latitude-data/duckdb-connector | Patch | | @latitude-data/mssql-connector | Patch | | @latitude-data/mysql-connector | Patch | | @latitude-data/postgresql-connector | Patch | | @latitude-data/snowflake-connector | Patch | | @latitude-data/sqlite-connector | Patch | | @latitude-data/test-connector | Patch | | @latitude-data/trino-connector | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR