Allow searching text for study name and description #1068
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix #965
Fix #1071
This PR adds some additional query options when using the text search bar, as well as establishes a framework for including more in the future. For this iteration, we have added plaintext searching for:
title
ofStudy
description
ofStudy
name
ofStudy
In order to accomplish this, our DSL query language has been updated to support using the SQL
ilike
function. This was a straightforward change for properties represented directly as columns on thestudy
table, but required some additional work for values stored in theannotation
column (JSON).There is also a new CLI command,
nmdc-server create-or-replace-nmdc-functions
which updates the custom NMDC functions used for ingest and search at a whim, rather than relying on the SQLAlchemy event system. See linked issue #1071 for some additional details.Updating these functions is also folded into the
nmdc-server migrate
command, which is invoked in our prestart script. This should populate the needed function when the app spins up.