search

CDRH / api

Codenamed "Apium": An API to access all public Center for Digital Research in the Humanities resources
https://cdrhdev1.unl.edu/api_frontend
MIT License
3 stars 1 forks source link

highlight field assumption discussion #66

Open jduss4 opened 7 years ago

jduss4 commented 7 years ago

Right now, highlighting is enabled by default on the text field only. If you want to disable highlighting entirely you may. If you want to enable highlighting for additional fields, you have to pass those parameters in specifically.

Would it be preferable to assume that any fields sent in the q search when highlighting is on should be returned in the highlighting results?

jduss4 commented 4 years ago

wait just kidding, I thought that we had discussed this at length somewhere but apparently it was on slack, not in that issue.

The idea we need to implement, it has been decided, is....

  1. If highlighting enabled but no fields specified, highlight any text fields in queries (text by default)
  2. If highlighting enabled and fields specified, only return for those fields
  3. If no highlighting enabled ignore