search

Islandora-Devops / islandora-starter-site

A Drupal site demonstrating Islandora features. Use it as a starting point for customization.
https://sandbox.islandora.ca
GNU General Public License v2.0
4 stars 16 forks source link

Fix EDTF Year processor settings #143

Closed joecorall closed 2 months ago

joecorall commented 3 months ago

Controlled_access_terms: the solr processor settings are of a new format

Without the settings changes to the new format, this will cause the processor to break and/or emit errors

Related Links

https://github.com/Islandora/controlled_access_terms/pull/113 https://github.com/Islandora/controlled_access_terms/pull/114 https://islandora.slack.com/archives/CM5PPAV28/p1712939510602939 https://github.com/Islandora/controlled_access_terms/issues/117

aOelschlager commented 2 months ago

Yesterday, I pulled this in to test and I'm still getting search_api processor errors. I tested this on a new isle-dc clone using this branch of the starter site. I used workbench to load demo content from the islandora demo objects repo. Screenshot from 2024-04-24 12-03-28 Screenshot from 2024-04-24 12-05-11 When I do a search this shows up in the logs. Screenshot from 2024-04-25 09-37-31 I'm going to update controlled access terms on my test site so I can utilize the recent pr and see if that takes care of these messages in the log.

aOelschlager commented 2 months ago

Updating controlled access terms seemed to fix that. I'm going test this one more time with the updated controlled access terms on a new starter site just to make sure but I think it should be good to go.

aOelschlager commented 2 months ago

There were no EDTF errors or messages in the logs with this pr and the most recent controlled access terms release. Could the lock file be updated to use that release or does that need to be submitted as a separate pr?

joecorall commented 2 months ago

Thanks @aOelschlager - updated controlled_access_terms