Open rmzelle opened 7 years ago
This thread is going to get very messy very quickly, I think. I think a set of Zotero guidance pages on a wiki in csl-evolution
Well, I've just been amending the first post. The main goal was to help myself keep track which existing issues can be implemented right away in Zotero and which ones need follow-up discussion, and to prepare a to-do list for Dan. We probably shouldn't have discussions here about the content of any of the issues, except if people disagree with their classification in the list above, but that can also take place in the issues themselves.
I'd suggest that https://github.com/citation-style-language/zotero-bits/issues/83, remap Zotero item type "podcast" to CSL "broadcast" instead of CSL "song" would be a good fit for Zotero 5.1
OK, we're just about ready to start doing this. @rmzelle, could you transfer this repo to me so that I can transfer it to the Zotero account, so that we can close issues as they're added?
@dstillman, to you personally?
Wouldn't it be easier to leave the repo in-place and just give you admin access to this repo? There are a number of issues that require changes on the CSL side.
To me personally so that I can then transfer it to the Zotero GitHub organization. Since you're not a member of the Zotero organization, you can't transfer it directly.
We want to be able to solicit schema changes (which will mostly just be JSON changes) and close issues in this repo via Zotero commits. It's not totally clear how the mechanics of that work if only I have push access to this repo, so it seemed easier to just have this in the Zotero account.
The GitHub documentation seems to have changed a bit recently, but
says "For example, including Closes example_user/example_repo#76 will close the referenced issue in that repository, provided you have push access to that repository."
So that suggests you should be able to auto-close tickets via commits if I give you push access to this repo. If that's your main concern, let's just give it a try, and I can help you transfer the repo if it doesn't work.
@dstillman, just gave you write access.
@bdarcus @rmzelle We've addressed everything CSL-related in this repository or moved them to another repo, so I think this one could be moved to the Zotero org at this point.
I'd like to start organizing the issues in repository a bit better so that we can more easily share them with the Zotero developers. Dan mentioned he's planning to mostly rely on our input regarding changes to the Zotero item type and field model.
So, below I plan to summarize the current issues, with special attention to those of high importance and/or a high level of consensus. I aim to highlight any areas where further discussion may be required. As a general resource, the current Zotero item model can be browsed at http://aurimasv.github.io/z2csl/typeMap.xml. It will be a work-in-progress, and feel free to chip in. Discussions specific to any given issue should take place in the issue itself.
There are usually up to three levels to each item field: the Zotero UI label, the Zotero internal field name, and, if the metadata is mapped to CSL, the corresponding CSL variable name. We will refer to these as the "label", "field", and "variable", respectively. For item types, we similarly will use "label", "Zotero item type", and "CSL item type", respectively.
1, add the "DOI" field (label = "DOI") to many more, if not all, Zotero item types, and map the field to the "DOI" CSL variable. It's not entirely clear if some item types should be exempted. (e.g. I've never seen a patent with a DOI)
2, add the "date" field (label = "Date") to the "podcast" Zotero item type, and map the field to the "issued" CSL date variable. See the "audioRecording" Zotero item type, which already has the "date" field, for reference.
4, add the "ISSN" field (label = "ISSN") to more Zotero item types, and map the field to the "ISSN" CSL variable. CSL 1.0.1 added the "ISSN" CSL variable. The "journalArticle", "magazineArticle", and "newspaperArticle" Zotero item types already have the field and are mapped to the CSL variable. Candidate Zotero item types are "blogPost", "book", "bookSection", "conferencePaper", "report", and "thesis".
5, add the "section" field (label = "Section") to the "journalArticle" Zotero item type, and map the field to the "section" CSL variable.
6, improve handling of place information. Zotero currently uses the "place" field (label = "Place"), which for all Zotero item types where it's used is mapped to both the "event-place" and "publisher-place" CSL variables. Some Zotero item types need both types of place information, so we suggest replacing the "place" field with new "eventPlace" and "publisherPlace" fields. The issue contains suggestions on which Zotero item types need which of these proposed new fields. How existing data should be migrated, and what the field and labels should be named, still needs to be worked out.
10, create new "PMID" (label ="PMID") and "PMCID" (label = "PMCID") fields to the "journalArticle" Zotero item type, and map the fields to their respective CSL variables, "PMID" and "PMCID". The issue discusses supporting other identifiers as well, and I recall Dan contemplating a more general solution, but these two are by far the most commonly requested. These new fields should be populated from the "extra" field, if possible, since Zotero translators have been saving this metadata there for quite a while, in either the old
{:PMCID: PMC4651100}
or newerPMCID: PMC4651100
format.Issues that need CSL changes:
12, add a field for the part number for e.g. the "journalArticle" Zotero item type. Probably requires a new CSL variable.
Issues that need more work on the exact requirements:
9, add fields to the "letter" Zotero item type (to map to the "event-place" (certain), and "medium" and "archive-place" CSL variables (not certain; need examples from style guides)).
11, add a field for the abbreviated version of the existing "Series" field, to map to the short form of "collection-title" CSL variable, for the "book" and "bookSection" Zotero item types. Need examples from style guides.