Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blacklight 7 SearchHelper Removal #62

Closed
Tracked by #22
ShanaLMoore opened this issue Aug 10, 2023 · 2 comments
Closed
Tracked by #22

Blacklight 7 SearchHelper Removal #62

ShanaLMoore opened this issue Aug 10, 2023 · 2 comments
Assignees
Milestone

Comments

@ShanaLMoore
Copy link

ShanaLMoore commented Aug 10, 2023

Upgrading blacklight to 7 removed the SearchHelper module, that GBH is using in their AMS::Export::CatalogSearch file.

We decided to time box our efforts to resolve issues to be as close as possible to .dassie/hyrax. If it proves to be too difficult we would make tickets that someone could address separately, while others continue on with the upgrade.

In this case we are noting that we need to figure out what needs to change due to the removal of Blacklight::SearchHelper.

PS. There is a spec for this that may help.

reference WIP turned into this ticket:
search_helper_catalog.rb

search_helper_catalog_spec.rb => currently skips running this test if future rails

run specs with DEPENDENCIES_NEXT=1 prepended to your commands

notes

dassie is a good reference point because they had to upgrade it from 2.9.6 yo hyrax 4 so there should be a lot of clues to follow.

Maybe helpful resources for us in this process:
Upgrade to Blacklight 7, Bootstrap 4 and Rails 6.0
Update koppie catalog controller for blacklight 7
blacklight Version 7.0.0 release notes

run spec like DEPENDENCIES_NEXT=1 rspec ./spec/services/ams/export/search/digital_instantiations_search_spec.rb:44

@ShanaLMoore
Copy link
Author

Look for TODO comments in main regarding catalog_search files and specs.

@jillpe jillpe added this to AMS / GBH Aug 15, 2023
@jillpe jillpe moved this to Ready for Development in AMS / GBH Aug 15, 2023
@jillpe jillpe added this to the Hyrax Upgrade milestone Aug 15, 2023
@ShanaLMoore
Copy link
Author

We replaced Blacklight::SearchHelper with a call to Hyrax::SearchService.

@github-project-automation github-project-automation bot moved this from Ready for Development to Done in AMS / GBH Aug 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

3 participants