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

Update Datacite JSON #3280

Merged
merged 3 commits into from May 17, 2024
Merged

Update Datacite JSON #3280

merged 3 commits into from May 17, 2024

Conversation

adam3smith
Copy link
Collaborator

Mainly to account for better publication metadata since Metadata Schema v. 4.4, also some additional improvements

Mainly to account for better publication metadata since Metadata Schema v. 4.4, also some additional improvements
Datacite JSON.js Outdated
@@ -173,14 +179,17 @@ function doImport() {
else if (contributor.nameType == "Personal") {
item.creators.push(ZU.cleanAuthor(contributor.name, role));
}
else {
item.creators.push({ lastName: contributor.name, creatorType: role, fieldMode: 1 });
else {
Copy link
Member

Choose a reason for hiding this comment

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

Trailing whitespace

Datacite JSON.js Outdated
else {
item.creators.push({ lastName: contributor.name, creatorType: role, fieldMode: 1 });
else {
item.creators.push({ lastName: contributor.name, creatorType: role, fieldMode: 1 });
Copy link
Member

Choose a reason for hiding this comment

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

Ditto

Datacite JSON.js Outdated Show resolved Hide resolved
Datacite JSON.js Outdated Show resolved Hide resolved
Datacite JSON.js Outdated Show resolved Hide resolved
Datacite JSON.js Outdated
Comment on lines 303 to 305
lastName: contributor.familyName,
firstName: contributor.givenName,
creatorType: role
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
lastName: contributor.familyName,
firstName: contributor.givenName,
creatorType: role
lastName: contributor.familyName,
firstName: contributor.givenName,
creatorType: role

Datacite JSON.js Show resolved Hide resolved
Datacite JSON.js Show resolved Hide resolved
adam3smith and others added 2 commits March 22, 2024 20:26
Co-authored-by: Abe Jellinek <jellinek@berkeley.edu>
@adam3smith
Copy link
Collaborator Author

@AbeJellinek I think this should be all set

@adam3smith
Copy link
Collaborator Author

@AbeJellinek -- currently all publishers/repositories import as [object Object] from Datacite. Could we get this merged? The CI just times out.

@AbeJellinek AbeJellinek merged commit c140b80 into zotero:master May 17, 2024
1 check failed
@AbeJellinek
Copy link
Member

Thanks - sorry for the delay.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants