-
Notifications
You must be signed in to change notification settings - Fork 0
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
Fix module according to feedback #22
Comments
All of the issues are now fixed. I will send the new test data to check them. |
I've sent the new test data to mr. Kvasnica. |
Time to update the export procedure and tests, then we can send next data batch. |
Email from mr. Kvasnica:
|
Disabled features allowed again. See 38b2ca1 for details. |
It will be necessary to add more of the postprocessing. Email feedback from Jaroslav Kvasnica:
<physicalLocation>
by se neměl používat pro elektronické dokumenty.Element
<url>
pak musí vést na místo, kde je dokument zpřístupněn (K4??). V elementu<relatedItem>
by další<location>
již nemělo být.Takto by to mělo správně vypadat. Pokud budmeme chtít přidat odkaz do K4, tak do elementu
<url>
a dát ho hned od<location>
u elementu
<genre>
je špatný obsah. Má tam být<mods:genre>electronic volume</mods:genre>
místo "elektronic title".Genre vždy odpovídá tomu, co je uvedeno jako ID u elementu
<mods>
element
<issuance>
sice umožňuje použít hodnotu "monografic", ale je lepší používat hodnoty "single unit" pro běžnou monografii a "multipart monograph" pro vícesvazkyjméno adresáře ve specifikace je "data" místo "original"
u info.xml chybí deklarace schéma
<info xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="http://www.ndk.cz/standardy-digitalizace/info11.xsd">
s tím souvisí i chybějící povinný element
<mainmets>
, který ty nemůžeš naplnit, ale možná by se měl dát prázdný, kvůli validaciThe text was updated successfully, but these errors were encountered: