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

2095-noOverwritingHbzId #2096

Merged
merged 3 commits into from
Nov 15, 2024
Merged

2095-noOverwritingHbzId #2096

merged 3 commits into from
Nov 15, 2024

Conversation

TobiasNx
Copy link
Contributor

@TobiasNx TobiasNx commented Nov 14, 2024

This PR introduces a conditional to stop overwriting existing hbzId.

Context:

There are old aleph zdb records that have zdb id informations
in their MAB data in 026-1 but not in 025z1.

Due to this in context of the migration to ALMA their HT number was not replaced by the ZDB ID. Also in the old aleph lobid mapping these records had no zdbId

e.g. see:
ALEPH: http://lobid.org/hbz01/HT019286510 // https://aleph.lobid.org/resources/HT019286510
ALMA: https://lobid.org/marcxml/990217879290206441 // https://lobid.org/resources/990217879290206441


To ensure that old links still work we added an enrichment of all records with zdbId that are part of the NZ based on the latest data from aleph data before the switch to ALMA as master.

Also we with ALMA we started to catch the ZDB ID from 035 not only when they have the prefix (DE-600) but also if (DE-599)ZDB.

This resulted in overwriting existing hbzId in the records with mapped ones. We want to adjust this mechanism so that only records WITHOUT zdbIdare enriched with their old hbzId

@TobiasNx TobiasNx requested review from acka47 and dr0i November 14, 2024 14:31
@dr0i
Copy link
Member

dr0i commented Nov 14, 2024

Build fails.

@TobiasNx TobiasNx assigned acka47 and dr0i and unassigned TobiasNx Nov 14, 2024
@dr0i dr0i removed their assignment Nov 14, 2024
@TobiasNx
Copy link
Contributor Author

I unassigne Adrian as reviewer, since he should only see that we introduced this change. Due to the discussion at metadaten.community.

@TobiasNx TobiasNx removed the request for review from acka47 November 15, 2024 13:48
@TobiasNx TobiasNx assigned dr0i and unassigned acka47 Nov 15, 2024
@TobiasNx
Copy link
Contributor Author

@dr0i can you merge and deploy since the automatic deployment is already through?

@dr0i dr0i merged commit 7605172 into master Nov 15, 2024
1 check passed
@dr0i dr0i deleted the 2095-noOverwritingHbzId branch November 15, 2024 14:21
@dr0i
Copy link
Member

dr0i commented Nov 15, 2024

Manually deployed. Data should be updated next Monday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants