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

Add documentation for testing lc call slips on staging #738

Merged
merged 1 commit into from
Mar 28, 2024

Conversation

sandbergja
Copy link
Member

Also, fix the link and formatting in the README's mailcatcher section.

Copy link
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.

Thanks @sandbergja

@christinach christinach merged commit 4dfd87a into main Mar 28, 2024
7 checks passed
@christinach christinach deleted the testing-lc-call-slips branch March 28, 2024 19:54
a problem.
1. You may also choose to run `bundle exec rake lib_jobs:create_csv_for_selector_comparison`, which
generates a CSV of all possible records from OCLC,
including ones that are typically filtered out like
Copy link
Contributor

Choose a reason for hiding this comment

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

It actually filters out the ones that don't meet the criteria for the generally_relevant? method. I was just saying that it used to be that the generally_relevant? method was where we excluded based on publication location, but now that decision is made in the relevant_to_selector? method.

So, it should exclude juvenile, audiobooks, etc., but will include US / UK / Canada publications.

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.

3 participants