Skip to content

feat(search): type aware custom filters#1067

Merged
ThibaudDauce merged 4 commits intomainfrom
feat/global-search/type-aware-custom-filters
May 4, 2026
Merged

feat(search): type aware custom filters#1067
ThibaudDauce merged 4 commits intomainfrom
feat/global-search/type-aware-custom-filters

Conversation

@abulte
Copy link
Copy Markdown
Contributor

@abulte abulte commented May 1, 2026

Fix ecolabdata/ecospheres#1089

This introduces a new config parameter typeKeys that lets downstream say if a custom filter is shared across types or not. If it's not shared, the background search for other types is skipped. This behaves similarly to native filters.

@abulte abulte marked this pull request as ready for review May 1, 2026 09:32
@abulte abulte requested a review from ThibaudDauce May 1, 2026 09:41
abulte added a commit to opendatateam/udata-front-kit that referenced this pull request May 1, 2026
Copy link
Copy Markdown
Contributor

@ThibaudDauce ThibaudDauce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a fan of the duplication between the v-if and the useSearchFilter but I don't have a better solution right now :-)

Comment thread datagouv-components/src/components/Search/GlobalSearch.vue Outdated
Comment thread datagouv-components/src/composables/useStableQueryParams.ts Outdated
@abulte abulte requested a review from ThibaudDauce May 4, 2026 08:50
@ThibaudDauce ThibaudDauce merged commit 9c1a87e into main May 4, 2026
8 checks passed
@ThibaudDauce ThibaudDauce deleted the feat/global-search/type-aware-custom-filters branch May 4, 2026 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

datagouv/components: Type-scoped custom filters in GlobalSearch

2 participants