Skip to content

Upgrade to Blacklight 9#5235

Merged
sandbergja merged 17 commits intomainfrom
blacklight9
Mar 24, 2026
Merged

Upgrade to Blacklight 9#5235
sandbergja merged 17 commits intomainfrom
blacklight9

Conversation

@sandbergja
Copy link
Copy Markdown
Member

@sandbergja sandbergja commented Oct 15, 2025

Mainly updates to accommodate changes to the API of Blacklight's MetadataFieldComponent.

sandbergja and others added 12 commits March 24, 2026 11:45
Currently, several tests are failing because JS hits an error before loading
all the necessary things (we need the fix to projectblacklight/blacklight#3724
to be released)
…'s new API

Co-authored-by: Christina Chortaria <christinach@users.noreply.github.com>
Co-authored-by: James R. Griffin III <jrgriffiniii@users.noreply.github.com>
Co-authored-by: Ryan Laddusaw <rladdusaw@users.noreply.github.com>


It seems that there are differences in our approach that we should handle at some point
It was removed upstream in projectblacklight/blacklight#3751,
and we agree with this removal.
@coveralls
Copy link
Copy Markdown

coveralls commented Mar 24, 2026

Coverage Status

coverage: 95.469% (+0.3%) from 95.142%
when pulling 13d9458 on blacklight9
into c435889 on main.

@sandbergja sandbergja changed the title Begin Blacklight 9 upgrade Upgrade to Blacklight 9 Mar 24, 2026
@sandbergja sandbergja marked this pull request as ready for review March 24, 2026 19:43
@christinach christinach self-requested a review March 24, 2026 19:56
Copy link
Copy Markdown
Member

@christinach christinach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sandbergja 🍄 thank you for all of your work on this upgrade! !!!

def remove_deftype(solr_parameters)
# Inlcuding a defType=lucene, as stock blacklight does, can cause our facet-only
# advanced search results to be empty
solr_parameters.delete('defType')
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻

describe 'defType' do
let(:blacklight_config) { CatalogController.blacklight_config }
it 'does not add a defType' do
# Stock blacklight adds defType=lucene, which causes empty results
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻

@sandbergja sandbergja merged commit 51be408 into main Mar 24, 2026
15 checks passed
@sandbergja sandbergja deleted the blacklight9 branch March 24, 2026 20:10
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.

4 participants