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

Reduce usage of EtikettMaker #2106 #2123

Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

Reduce usage of EtikettMaker #2106 #2123

wants to merge 21 commits into from

Conversation

TobiasNx
Copy link
Contributor

@TobiasNx TobiasNx commented Jan 10, 2025

I migrated some label maps to fix and use them via lookup as filemaps there.
Also I deleted all unnecessary labels that already were provided by the fix.

While doing that I adjusted the lobid uri for collections to http (since all lobid uris we use are http) and fixed an error while mapping marc relator infos.

@TobiasNx TobiasNx changed the title 2106-marcRel Reduce usage of EtikettMaker #2106 Jan 10, 2025
@TobiasNx TobiasNx requested a review from dr0i January 10, 2025 15:14
@TobiasNx
Copy link
Contributor Author

TobiasNx commented Jan 13, 2025

@dr0i can you have a look at this? I reduced all Ettiket Maker usage to the context and labels for undefined uris.

@TobiasNx TobiasNx marked this pull request as ready for review January 13, 2025 11:18
@@ -109,7 +109,7 @@
"label" : "Mystik & Aszese des 16.-19. Jahrhunderts [Nationallizenz]",
"type" : [ "Collection" ]
}, {
"id" : "https://lobid.org/collections#NLZ",
"id" : "http://lobid.org/collections#NLZ",
"type" : [ "Collection" ],
Copy link
Member

Choose a reason for hiding this comment

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

If possible we should stick to https. Even more so if this was so before. Or do you have an argument that support your deliberately switching this?

Copy link
Contributor Author

@TobiasNx TobiasNx Jan 13, 2025

Choose a reason for hiding this comment

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

I was remembering: #1343 (comment)
That we had to switch back to http instead of https.
And thought this was an error on our side.

Then I checked:

aleph.lobid.org/resources/TT050372336.json

<combine name="http://purl.org/lobid/lv#inCollection" value="https://lobid.org/collections#${a}">
<data source="@078e1aNotZDB" name="a">
<regexp match="(.*)" format="${1}" />
</data>
</combine>

It seems to have used https for the collection URIs.

I roll back this change then.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I adjusted it as requested.

},
"role" : {
"id" : "http://id.loc.gov/vocabulary/relators/http://id.loc.gov/vocabulary/relators/edt",
"label" : "edt"
Copy link
Member

Choose a reason for hiding this comment

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

ah, this PR also seems to fix a broken id 👍

@dr0i dr0i assigned TobiasNx and unassigned dr0i Jan 13, 2025
@TobiasNx TobiasNx requested a review from dr0i January 13, 2025 15:56
@TobiasNx TobiasNx assigned dr0i and unassigned TobiasNx Jan 13, 2025
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