Skip to content

JSONSchema fixes#59

Merged
mfenner merged 5 commits intodatacite:masterfrom
lnielsen:lang-fix
Aug 5, 2019
Merged

JSONSchema fixes#59
mfenner merged 5 commits intodatacite:masterfrom
lnielsen:lang-fix

Conversation

@lnielsen
Copy link
Copy Markdown
Contributor

@lnielsen lnielsen commented Jul 25, 2019

This builds on #55 and found as part of merging inveniosoftware/datacite#48

See the individual commits.

  • Add lang attribute to creators and contributors
    • Schema v4.2 added the xml:lang attribute to creators, contributors and publisher properties. In the XML schema this property is on the creatorName/contributorName. This commit is not adding the lang attribute to the publisher property as the solution requires further discussion (#60).
  • Require the polygonPoints in geoLocationPolygons
    • According to the v4.2 documentation (18.4.1), the polygon points is a required property if geoLocationPolygons are specified.
  • Remove agency property from JSONSchema
    • The agency property is not part of the official v4.2 documentation and thus should not be a required property.

tmorrell and others added 5 commits July 25, 2019 13:31
Schema v4.2 added the xml:lang attribute to creators, contributors and
publisher properties. In the XML schema this property is on the
creatorName/contributorName. This commit is not adding the lang
attribute to the publisher property as the solution requires further
discussion.
According to the v4.2 documentation (18.4.1), the polygon points is a
required property if geoLocationPolygons are specified.
The agency property is not part of the official v4.2 documentation and
thus should not be a required property.
@mfenner
Copy link
Copy Markdown

mfenner commented Aug 5, 2019

LGTM

@mfenner mfenner merged commit 3167813 into datacite:master Aug 5, 2019
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.

3 participants