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

Summary for Zotero devs #73

Open
rmzelle opened this issue Dec 30, 2016 · 9 comments
Open

Summary for Zotero devs #73

rmzelle opened this issue Dec 30, 2016 · 9 comments
Labels

Comments

@rmzelle
Copy link
Collaborator

rmzelle commented Dec 30, 2016

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.

  • DOI #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)
  • Date (for podcast) #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.
  • ISSN #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".
  • Section (for journalArticle) #5, add the "section" field (label = "Section") to the "journalArticle" Zotero item type, and map the field to the "section" CSL variable.
  • Migrate Place to either Event Place or Publisher Place #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.
  • PMID and PMCID (for journalArticle) #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 newer PMCID: PMC4651100 format.

Issues that need CSL changes:

Issues that need more work on the exact requirements:

  • Pages (for letter) #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)).
  • Series Abbr (for book and bookSection) #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.
@bwiernik
Copy link
Collaborator

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

@rmzelle
Copy link
Collaborator Author

rmzelle commented Mar 10, 2017

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.

@bwiernik
Copy link
Collaborator

bwiernik commented Dec 2, 2018

I'd suggest that #83, remap Zotero item type "podcast" to CSL "broadcast" instead of CSL "song" would be a good fit for Zotero 5.1

@dstillman
Copy link
Member

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?

@rmzelle
Copy link
Collaborator Author

rmzelle commented Feb 27, 2020

@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.

@dstillman
Copy link
Member

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.

@rmzelle
Copy link
Collaborator Author

rmzelle commented Feb 28, 2020

The GitHub documentation seems to have changed a bit recently, but

https://help.github.com/en/enterprise/2.16/user/github/managing-your-work-on-github/closing-issues-using-keywords#closing-an-issue-in-the-same-repository

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.

@rmzelle
Copy link
Collaborator Author

rmzelle commented Feb 28, 2020

@dstillman, just gave you write access.

@bwiernik
Copy link
Collaborator

@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.

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

No branches or pull requests

3 participants