Join GitHub today
GitHub is home to over 36 million developers working together to host and review code, manage projects, and build software together.
Sign upAdd translator Landesbibliographie Baden-Wurttemberg.js #1774
Conversation
zuphilip
and others
added some commits
Nov 1, 2018
adam3smith
requested changes
Nov 1, 2018
A couple of questions&requests, looks good otherwise. Thanks! |
"items": [ | ||
{ | ||
"itemType": "journalArticle", | ||
"title": "Überfordert : Mozart: Don Giovanni : Mannheim, Nationaltheater", |
This comment has been minimized.
This comment has been minimized.
adam3smith
Nov 1, 2018
Collaborator
Could you remove spaces before colons in titles and update tests?
"fieldMode": 1 | ||
} | ||
], | ||
"date": "2000///(2001); 2001/02(2003) -", |
This comment has been minimized.
This comment has been minimized.
adam3smith
Nov 1, 2018
Collaborator
Can we clean this? I'm not even sure how Zotero would end up parsing this date (though I think just as 2000?).
This comment has been minimized.
This comment has been minimized.
zuphilip
Nov 1, 2018
Author
Collaborator
No, it is more complicated because here the whole series is meant. So
Erscheinungsverlauf: | 1972/73 - 1999/2000(2001); 2001/02(2003) -
means that there was a report for 1972/1973, ..., and another report for 1999/2000 published in 2001, but no report in 2000/2001 (the note explained why). Then from 2001/2002 in 2003 it started again normally.
How about moving the date in this case to a note? Deleting would be okay also for me, because these report (series) are not my main focus here.
This comment has been minimized.
This comment has been minimized.
adam3smith
Nov 1, 2018
Collaborator
I'd still pick a date to import, maybe the first one? Putting the rest into notes sounds right.
"title": "Rechenschaftsbericht ... des Rektors", | ||
"creators": [ | ||
{ | ||
"lastName": "Universität ", |
This comment has been minimized.
This comment has been minimized.
adam3smith
Nov 1, 2018
•
Collaborator
This is as <Mannheim>
in the RIS export and then apparently ignored by Zotero. I'm suspecting because that was your search term? I think it's save to just clean pointy brackets from the RIS completely?
This comment has been minimized.
This comment has been minimized.
zuphilip
Nov 1, 2018
Author
Collaborator
This comes from
AU - Universität <Mannheim>
which is catalogued as a "Körperschaft" in the original. Maybe we could move it to a contributor
but I don't know how to identifies these cases. Thoughts?
This comment has been minimized.
This comment has been minimized.
adam3smith
Nov 1, 2018
Collaborator
Sorry, I had seen that but markdown swallowed the relevant bit. I think we should leave it as author, which seems fine to me, but we should filter out the pointy brackets (< >) so that it actually imports as "Universität Mannheim" and not just "Universität "
zuphilip
added some commits
Nov 1, 2018
This comment has been minimized.
This comment has been minimized.
Please have a look at the new version. |
zuphilip
added
the
New Translator
label
Nov 1, 2018
adam3smith
merged commit 8f64313
into
zotero:master
Nov 4, 2018
1 check passed
This comment has been minimized.
This comment has been minimized.
Great, thanks! |
zuphilip commentedNov 1, 2018
No description provided.