-
Notifications
You must be signed in to change notification settings - Fork 7
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
2074-introxOpacLinks #2075
2074-introxOpacLinks #2075
Conversation
Only CZ resources work with alma mms id links.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm afraid we had this discussion before, but why so complicated? Why not just walk through the priority list (hbz ID, ZDB ID, MMS ID, ISSN) and use the first one found?
Also, note that I said "found"; it may be possible for a library not to have OPAC links of all types defined. Due to this uncertainty, I would not feel comfortable with the proposed solution.
I think we skip those without any matching opac links. Any suggestion how to change the mapping for those? |
Of course, there should be no non-functional links. Are you aware of any?
I'm not sure I understand. Yes, of course they are skipped. But then there may be no link remaining.
I can work on an alternate proposal if that's what you're suggesting. |
Not sure about non functional links, but I checked if only digibib libraries have almaMmsIds, hbzId and zdbId links they do. In my opinion we can change the order then. |
I redid this PR and opened a new PR #2076 |
Replaced by #2076 |
Resolves #2074. Needed for hbz/lookup-tables#56
Only CZ resources work with alma mms id links in introx, NZ/Verbund resources need hbzId or zdbId as base.