-
Notifications
You must be signed in to change notification settings - Fork 3
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
incompatibility with Blacklight >= 5.16.0 #81
Comments
Have you tried moving it into an initializer there? |
Can you point to an example? An initializer in Or something like this, from |
Yep, exactly like that. |
I updated one of my blacklights to 6 today and put in overrides for blacklight-maps helpers that were calling the old catalog paths, etc. Should I try forking and offering stuff back or is blacklight-maps for 6 already getting worked on somewhere? |
I've done some initial work on it through the process of updating my BL apps to 6. Was hoping to have 6-friendly release ready for sometime late next week (by 6/3 or so). |
PR for BL-6.*-friendly release submitted: #82. |
When attempting to use blacklight-maps v0.4.0 with blacklight >= 5.16.0, you get an error
uninitialized constant Blacklight::Configuration (NameError)
when starting your application.Here's the full trace:
Blacklight::Maps::Engine
attempts to set some default configuration settings, which is what causes the error.(In 5.16.0, the
Blacklight::Configuration
module was moved toapp/models/blacklight/configuration.rb
.)There may be other issues related to using Blacklight >= 5.16.0, but this one is a major blocker to further work on bringing this project up to date with Blacklight 6.
@mejackreed @cbeer any ideas on where to move the default configuration stuff?
The text was updated successfully, but these errors were encountered: