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

Adjust license mapping to map 540 Open Access infos #2103 #2104

Merged
merged 1 commit into from
Nov 29, 2024

Conversation

TobiasNx
Copy link
Contributor

Improved the mapping. To fetch 540 and not only 542 for licensing info.

We could discuss, if we also intend to catch licensing info from 856 but I am not sure yet. And I have no example.

Copy link
Contributor

@acka47 acka47 left a comment

Choose a reason for hiding this comment

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

+1

@@ -93,6 +93,10 @@
} ],
"extent" : "1 online resource (194 pages) : digital file(s).",
"abstract" : [ "2012 wurde das Netzwerk-Projekt Landeskunde Nord ins Leben gerufen, dessen Ziel es ist, Forschung und Lehre zur Landeskunde des Deutschen als Fremdsprache, insbesondere in den nordischen Ländern, voranzutreiben. Das vorliegende Buch, Perspektive Nord, knüpft an den 2013 erschienenen Band Landeskunde Nord an und bietet Reflexionen wissenschaftstheoretischer, fachdidaktischer und inhaltlicher Aspekte, die für die Gestaltung der Landeskundelehre bzw. des Landeskundeunterrichts relevant sind. In ihrer Vielfalt sind die hier vorgelegten Konzepte und Ideen vor allem modernen kulturwissenschaftlichen Ansätzen verpflichtet, knüpfen an international geführte Diskussionen an und haben auch deshalb über die nordischen Länder hinaus Bedeutung. Dabei setzen die Autorinnen und Autoren auf autonomes und lebenslanges Lernen." ],
"license" : [ {
"id" : "https://creativecommons.org/licenses/by-nc-nd/4.0/legalcode",
"label" : "creativecommons.org/licenses/by-nc-nd/4.0/legalcode"
Copy link
Contributor

Choose a reason for hiding this comment

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

The label could be better. Apparently, we are currently not showing the field in the lobid UI, anyway. We will improve the labels when/if we add the field in the UI.

@TobiasNx TobiasNx assigned dr0i and unassigned acka47 Nov 28, 2024
@TobiasNx TobiasNx requested a review from dr0i November 28, 2024 13:46
@dr0i dr0i merged commit f2ec134 into master Nov 29, 2024
1 check passed
@dr0i dr0i deleted the 2103-fetch540OpenAccessInfo branch November 29, 2024 08:32
@acka47 acka47 linked an issue Nov 29, 2024 that may be closed by this pull request
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.

License from 540/542 is not transformed
3 participants