-
Notifications
You must be signed in to change notification settings - Fork 5
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
i3338 non-search param deprecation (blacklight deprecations, part 2) #3339
Conversation
6103f29
to
b2e70b3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sandbergja I noticed a padding issue in the Publication date facet, where the range limit is. Can you please check?
The new release fixes the search_state_fields deprecation warning
8f9a531
to
11ba58e
Compare
@christinach Thanks for catching this! Can you re-review? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @sandbergja !
Cribbing from @sandbergja work [here](pulibrary/orangelight#3339)
* Upgrading to blacklight 7.33 On the way to 8.0 * Updating config.search_state_fields in Blacklight Cribbing from @sandbergja work [here](pulibrary/orangelight#3339) * Removing render_search_bar in favor of Blacklight::SearchBarComponent Cribbing from @sandbergja work pulibrary/orangelight@330671e * Adding a to the search parameters It can be a part of the parameters on the error page * Removing BlacklightRangeLimit::ViewHelperOverride call it has been removed in the latest BlacklightRangeLimit version cribbing from @sandbergja pulibrary/orangelight@f3ddf37
Closes #3338