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

Adding sortable call number to the output for bib items #560

Merged
merged 1 commit into from
Sep 24, 2019

Conversation

carolyncole
Copy link
Member

@carolyncole carolyncole commented Sep 16, 2019

This allows the locator to find the location of the item
Including lcsort to do most of the legwork, but utilizing some custom code to pick up the items lcsort misses

See pulibrary/lcsort#21 for a list of items lcsort misses.

This should fix: pulibrary/voyager_helpers#104

    This allows the locator to find the location of the item
    Including lcsort to do most of the legwork, but utilizing some custom code to pick up the items lcsort misses
carolyncole added a commit to pulibrary/firestone_locator that referenced this pull request Sep 16, 2019
…rent locator generates

Some may be more accurate than the old code, based on the fact that the normalized version from voyager does not look normalized.

Added a PR into mrc_liberation pulibrary/bibdata#560 for adding the sortable call no.

Once that exists we need to change the SQL table, and use the new fields and new table.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 79.307% when pulling 35a215e on normal_call into c579636 on master.

@tampakis tampakis merged commit 44e8a3e into master Sep 24, 2019
@tampakis tampakis deleted the normal_call branch September 24, 2019 21:15
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.

Add the normalized call number to items
3 participants