Skip to content
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

Filter invalid search results #232

Merged
merged 3 commits into from
Nov 27, 2024
Merged

Filter invalid search results #232

merged 3 commits into from
Nov 27, 2024

Conversation

eikek
Copy link
Member

@eikek eikek commented Oct 25, 2024

This implements filtering for invalid results (where namespace doesn't exist or doesn't resolve to an existing namespace entity). It uses an additional solr "filter query", closes #224 therefore as well.

@eikek eikek changed the base branch from main to search-query-refactor October 25, 2024 13:04
@eikek eikek force-pushed the filter-invalid-search-results branch 2 times, most recently from 363899f to 6fed0d8 Compare October 25, 2024 13:41
@eikek eikek marked this pull request as ready for review October 25, 2024 13:52
@eikek eikek requested a review from a team as a code owner October 25, 2024 13:52
@eikek eikek marked this pull request as draft October 25, 2024 13:52
Base automatically changed from search-query-refactor to main October 28, 2024 07:07
@eikek eikek force-pushed the filter-invalid-search-results branch 5 times, most recently from 7d1dbbc to c91d8c2 Compare October 30, 2024 10:09
@eikek eikek changed the title wip: Filter invalid search results Filter invalid search results Oct 30, 2024
@eikek eikek added the chore Maintenance work label Oct 30, 2024
@eikek eikek marked this pull request as ready for review October 30, 2024 12:01
eikek added 3 commits October 31, 2024 07:06
This will select only entities, where a `namespace` property exists.
Additionally, it selects only projects that have an existing
`createBy` property.

This doesn't filter out results, where a subquery would return no
results.
A namespace is a reference to another entity. If this doesn't exist,
the results are not valid and are therefore not selected to be
returned from a user query
@eikek eikek force-pushed the filter-invalid-search-results branch from c91d8c2 to 8538e1a Compare October 31, 2024 06:06
@eikek eikek added fix A fix to a bug and removed chore Maintenance work labels Nov 27, 2024
@eikek eikek merged commit 50a0101 into main Nov 27, 2024
3 checks passed
@eikek eikek deleted the filter-invalid-search-results branch November 27, 2024 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix A fix to a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants