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

Fetch pubs from zotero on home instead of manual curation #1267

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

dannon
Copy link
Member

@dannon dannon commented Apr 1, 2022

Before this is merged, we probably need to clean up publication details a bit on zotero and pick precisely which ones we'll use.

Right now I'm using:

                    {{ item.meta.creatorSummary }}, 
                    {{ item.data.journalAbbreviation ? item.data.journalAbbreviation : "preprint" }},
                    {{ item.data.date }}.

So, creatorSummary, journalAbbreviation, and date should be carefully added. We can add more if we want. It looks like journalAbbreviation isn't always correct or is sometimes missing currently, so we'll need to update those records.

Another option is to just hit crossref for each doi and request a proper citation format, as mentioned before. This wouldn't be a lot of extra work, but I think we wanted to use this current approach to force zotero to be curated more.

Drops 'New Platforms' box as well, since we've decided to roll that into news when appropriate.

@dannon
Copy link
Member Author

dannon commented Apr 1, 2022

(testing fixes for the accessibility test failure, which appears to be related to a slow (or fast) load..)

@beatrizserrano
Copy link
Member

Thank you, @dannon! I've updated the library to automatically get some authors that were missing (field Creator) so that should work fine now. However, the Journal Abbreviation is not retrieved automatically (or I don't know how to do it) :(

@dannon
Copy link
Member Author

dannon commented Apr 4, 2022

@beatrizserrano You know more than I do about Zotero :) Is there another field that's more appropriate you'd like to use?

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.

2 participants