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

Store callnumber if record is 'In the Library' #465

Merged
merged 1 commit into from
Apr 11, 2023
Merged

Conversation

ajkiessl
Copy link
Contributor

@ajkiessl ajkiessl commented Apr 11, 2023

Closes #461

Replaced online? method in holdings.rb with not_in_the_library? method that checks if the record's access_facet includes 'In the Library'. If a record is not in the library it will not have a callnumber. On the other hand, anything (including records that are also online) that is in the library will have a callnumber. Added some tests to test a couple different variations of access_facets.

…ds that checks if the record's 'access_facet' includes 'In the Library'. If a record is not in the library it will not have a callnumber. On the other hand, anything (including records that are also online) that is in the library will have a callnumber. Added some tests to test a couple different variations of 'access_facets'.
@banukutlu banukutlu added this to the 1.2.x milestone Apr 11, 2023
Copy link
Contributor

@banukutlu banukutlu left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks Alex!

@banukutlu banukutlu merged commit ef97a4a into main Apr 11, 2023
@banukutlu banukutlu deleted the 461-in-lib-call-num branch April 11, 2023 17:44
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.

700987 has an LC call # but no keymap_struct
2 participants