Skip to content

Enable advanced search + support its use of Solr's JSON Query DSL - #103

Open
rshiggin wants to merge 4 commits into
mainfrom
arc-153-advanced-search
Open

Enable advanced search + support its use of Solr's JSON Query DSL#103
rshiggin wants to merge 4 commits into
mainfrom
arc-153-advanced-search

Conversation

@rshiggin

@rshiggin rshiggin commented Sep 9, 2026

Copy link
Copy Markdown
Member

To do:

  • Prevent /advanced relative link from reverting to / home page.
  • add at least one link for initial access to Advanced Search. Other style changes on its page at /advanced?

@rshiggin rshiggin changed the title Significant changes to support advanced search use of Solr's JSON Query DSL Enable advanced search + support its use of Solr's JSON Query DSL Sep 9, 2026
@rshiggin

Copy link
Copy Markdown
Member Author

We need to override to remove "More Options" button for Advanced Search -- at least in the interim. Found in component gems/blacklight-8.12.3/app/components/blacklight/search_bar_component.html.erb

more-options-button

@rshiggin

Copy link
Copy Markdown
Member Author

Revise _home.html.erb to match prod link for Advanced Search.
home-advanced-search

@rshiggin

rshiggin commented Sep 11, 2026

Copy link
Copy Markdown
Member Author

Overriding More Options button in the search bar is more complicated than it should be, but because both Arclight and Blacklight contain search bar components it looks like it's necessary.

Need um partials of the following files:

  • (blacklight) search_bar_component.rb
  • (arclight) search_bar_component.rb
  • (arclight) search_bar_component.html.erb
  • catalog_controller.rb -- config.index.search_bar_component = UmSearchBarComponent

… advanced search by Blacklight in the nav search bar
@rshiggin
rshiggin force-pushed the arc-153-advanced-search branch from 0aa5e55 to b3cbc37 Compare September 11, 2026 20:21
@rshiggin
rshiggin marked this pull request as ready for review September 11, 2026 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant