pulibrary / orangelight

PUL Blacklight Project (Ruby 3.1.0, Rails 7.1.4)
21 stars 7 forks source link

Refactor advanced search using the built-in advanced search BL form #3412

Closed christinach closed 1 year ago

christinach commented 1 year ago

The built-in advanced search form is included in BL 7 and BL 8. Things to consider:

  1. The queries will turn from q1, q2, q3 to [clause0], [clause1], [clause2]
  2. It uses view components rather the view partials.
  3. It will need a lot of work to make the form look as it does now.
  4. It uses the json query DSL. This is the new way BL communicates with SOLR. This will require changes to the catalog controller.

Acceptance criteria

maxkadel commented 1 year ago

Still to do:

maxkadel commented 1 year ago

Hey team! Please add your planning poker estimate with Zenhub @regineheberlein @kevinreiss @christinach @rladdusaw @sandbergja

sandbergja commented 1 year ago

I was able to follow these procedures to get advanced search working with the blacklight .internal_test_app: https://code4lib.slack.com/archives/C54TB5WDQ/p1687452502967029

These don't work for orangelight -- I wonder if it's because we also have a complex default query that we also send.

kevinreiss commented 1 year ago

Form is partially working, but need to work through these scenarios.

  1. Add support for Boolean operators (they are there now but don't work)
  2. Pivot facets are failing when you edit an active search. Possibly related to nil values based on work on the numismatics form for the BL8 upgrade.