Skip to content

feat(dpav-3442): include organisation name in product free-text search - #95

Open
kkinformed wants to merge 2 commits into
developfrom
feature/DPAV-3442-search-organisation-name
Open

kkinformed wants to merge 2 commits into
developfrom
feature/DPAV-3442-search-organisation-name

Conversation

@kkinformed

@kkinformed kkinformed commented Oct 28, 2026 •

Copy link
Copy Markdown
  • Free-text search (textSearch) now also matches the owning organisation’s name, so searching “Bristol” finds Bristol City Council’s products.
  • OPA: organisation.name added to text_search_fields in product_access.rego.
  • ProductField: ORGANISATION_NAME now uses a new FILTER_SORT_SEARCHED mode. It is included in free-text search, but a single-value filter on it still matches the whole name (EQ), as before.
  • ProductSearchCriteriaFactory: the default filter operator now depends on a new isContainsByDefault() flag rather than on whether the field is searchable.

Security scan is failing due to dependency snappy-java on develop, which if indirectly brought by spring-kafka - not related to this PR

@github-actions

github-actions Bot commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

OSS Checks Passed

All tracked OSS checks passed in this run.

Total Files Passed Failed Score
15 15 0 100%

Results from commit 039bb57, view the full job summary for detailed results.

♻️ This comment has been updated with latest results.

@kkinformed
kkinformed marked this pull request as ready for review October 28, 2026 10:45
@kkinformed
kkinformed force-pushed the feature/DPAV-3442-search-organisation-name branch from d82df8c to 039bb57 Compare October 28, 2026 10:58
@sonarqubecloud

Copy link
Copy Markdown

This branch has not been deployed

No deployments
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.

1 participant