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

Overcome backwards xl-id form by handling both #422

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

niklasl
Copy link
Member

@niklasl niklasl commented Feb 27, 2023

This adds the "normal" form (used in XL) as a sameAs to generated records if they are to have the backwards form mistakenly minted by this repository.

An upper timestamp per dataset is used to check for if its member records are to have the backwards form.

(Eventually we want to "garbage collect" this backwards form from XL, to ensure they don't "squat" on XL id:s in the future.)

This adds the "normal" form (used in XL) as a sameAs to generated
records if they are to have the backwards form mistakenly minted by this
repository.

An upper timestamp per dataset is used to check for if its member
records are to have the backwards form.

(Eventually we want to "garbage collect" this backwards form from XL, to
ensure they don't "squat" on XL id:s in the future.)
@niklasl niklasl force-pushed the feature/backwards_xl_id branch from fcd147d to 195d085 Compare February 28, 2023 13:38
@niklasl niklasl marked this pull request as ready for review February 28, 2023 15:17
@niklasl niklasl requested review from kwahlin, klngwll and olovy February 28, 2023 15:17
lxltools/datacompiler.py Outdated Show resolved Hide resolved
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.

2 participants