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

2074-introxOpacLinks #2075

Closed
wants to merge 2 commits into from
Closed

2074-introxOpacLinks #2075

wants to merge 2 commits into from

Conversation

TobiasNx
Copy link
Contributor

@TobiasNx TobiasNx commented Sep 25, 2024

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.

Copy link
Member

@blackwinter blackwinter left a 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.

@blackwinter blackwinter assigned TobiasNx and unassigned blackwinter Sep 25, 2024
@TobiasNx
Copy link
Contributor Author

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?
i would be okay with this if the lookup tables are cleaned up and those libraries that just use alma mms id have no hbzId opac link and no zdbId alma link in the lookup table.

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?

@blackwinter
Copy link
Member

i would be okay with this if the lookup tables are cleaned up and those libraries that just use alma mms id have no hbzId opac link and no zdbId alma link in the lookup table.

Of course, there should be no non-functional links. Are you aware of any?

I think we skip those without any matching opac links.

I'm not sure I understand. Yes, of course they are skipped. But then there may be no link remaining.

Any suggestion how to change the mapping for those?

I can work on an alternate proposal if that's what you're suggesting.

@TobiasNx
Copy link
Contributor Author

i would be okay with this if the lookup tables are cleaned up and those libraries that just use alma mms id have no hbzId opac link and no zdbId alma link in the lookup table.

Of course, there should be no non-functional links. Are you aware of any?

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.

@TobiasNx
Copy link
Contributor Author

I redid this PR and opened a new PR #2076

@TobiasNx
Copy link
Contributor Author

Replaced by #2076

@TobiasNx TobiasNx closed this Sep 25, 2024
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 adjustment for Introx opac links
2 participants