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

Rm auto-inserted solr availability subjects ("Accessible book", "Protected DAISY") #6596

Merged
merged 1 commit into from
May 31, 2022

Conversation

cdrini
Copy link
Collaborator

@cdrini cdrini commented May 27, 2022

Apparently these were being injected into solr even if they weren't on the record itself ! Knock these out.

Technical

Testing

Trusting unit tests that nothing broke crazily here ; assuming the fields will no longer be there now!

Screenshot

Stakeholders

@cdrini cdrini marked this pull request as ready for review May 27, 2022 17:17
No more "Accessible book" or "Portected DAISY" subjects
@cdrini cdrini force-pushed the fix/rm-bad-solr-subjects branch from e3a0dbd to 23328da Compare May 27, 2022 20:15
@cdrini cdrini added Module: Solr Updater Code relating to live edits being reflected into solr Module: Solr Issues related to the configuration or use of the Solr subsystem. [managed] Theme: Search Issues related to search UI and backend. [managed] and removed Module: Solr Updater Code relating to live edits being reflected into solr Module: Solr Issues related to the configuration or use of the Solr subsystem. [managed] labels May 27, 2022
@mekarpeles mekarpeles merged commit 4f1d954 into internetarchive:master May 31, 2022
@cdrini cdrini deleted the fix/rm-bad-solr-subjects branch June 1, 2022 01:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Theme: Search Issues related to search UI and backend. [managed]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants