denshoproject / ddr-public

Web UI for publishing DDR collections.
Other
1 stars 1 forks source link

search on creators.namepart and persons is broken #199

Open gjost opened 1 year ago

gjost commented 1 year ago

Mods to ddrpublic on the 199-creators-persons branch:

densho-elastictools expects searches to always be based on a fulltext field. This branch enables searches based on creators.namepart and persons fields instead of fulltext. Unfortunately these fields are tokenized (elasticsearch_dsl.Keyword instead of elasticsearch_dsl.Text) and can't be mixed with a fulltext search without some more refactoring.

The team thought this change was going in a wrong direction, that users would expect to be able to type names and get results from title, description, and other fields along with results from creators.namepart, all using fulltext search. To make that work, we will need to create a separate elasticsearch_dsl.Text field that contains the text from creators.namepart and make that searchable.

199-creators-persons branch will be kept in place but further work will be on 199-search-hidden.

gjost commented 1 year ago
gjost commented 1 year ago

creators and persons are now searched using fulltext searching via the search_hidden field.

Fixed in ddr-public commit a03701e, merged in to develop branch, pulled to ddrstage.densho.org

gjost commented 1 year ago

Some ideas: