Skip to content

feat(dashboards): Support multiple filters on dashboards list endpoint#109973

Merged
DominikB2014 merged 1 commit intomasterfrom
dominikbuszowiecki/dain-1284-update-backend-to-allow-multiple-filters
Mar 6, 2026
Merged

feat(dashboards): Support multiple filters on dashboards list endpoint#109973
DominikB2014 merged 1 commit intomasterfrom
dominikbuszowiecki/dain-1284-update-backend-to-allow-multiple-filters

Conversation

@DominikB2014
Copy link
Contributor

Support multiple filter query params on the dashboards list endpoint so we can toggle prebuilt dashboards while also filtering by other fields like owned or shared.

Previously filter only accepted a single value via an if/elif chain, making combinations like ?filter=owned&filter=excludePrebuilt impossible. This switches from query_params.get("filter") to query_params.getlist("filter") and applies each filter sequentially (AND logic). A single ?filter=owned behaves identically to before, so this is fully backwards compatible.

Refs DAIN-1284

Replace the single-value `filter` query param with `getlist("filter")`
so the same param accepts multiple values. Filters are applied
sequentially (AND logic), enabling combinations like
`?filter=owned&filter=excludePrebuilt`.

Backwards compatible — a single `?filter=owned` behaves identically.

Refs DAIN-1284
Co-Authored-By: Claude <noreply@anthropic.com>
@linear-code
Copy link

linear-code bot commented Mar 5, 2026

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 5, 2026
@DominikB2014
Copy link
Contributor Author

@sentry review

@DominikB2014 DominikB2014 marked this pull request as ready for review March 5, 2026 21:20
@DominikB2014 DominikB2014 requested a review from a team as a code owner March 5, 2026 21:20
@DominikB2014 DominikB2014 merged commit 87cfae3 into master Mar 6, 2026
58 checks passed
@DominikB2014 DominikB2014 deleted the dominikbuszowiecki/dain-1284-update-backend-to-allow-multiple-filters branch March 6, 2026 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants