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

Add UNZ Print Archive.js #1272

Merged
merged 4 commits into from
Jul 23, 2021
Merged

Add UNZ Print Archive.js #1272

merged 4 commits into from
Jul 23, 2021

Conversation

owcz
Copy link
Contributor

@owcz owcz commented Mar 15, 2017

Translator for UNZ.org periodical archive. Really powerful—works on every page (save for individual books without reviews, which don't have useful metadata and for which users are better off clicking through to their Internet Archive source).

@zuphilip zuphilip added the New Translator Pull requests for new translators label Nov 12, 2017
@owcz
Copy link
Contributor Author

owcz commented Jul 8, 2018

ready for review

  • removed Framework per Deprecate/integrate FW #1277
  • multi support for issue pages, search
  • only covers the print archive (unz.com/print, formerly unz.org) and not the new conservative blog unz.com

@owcz owcz changed the title Add UNZ.org.js Add UNZ Print Archive.js Jul 8, 2018
@owcz
Copy link
Contributor Author

owcz commented Oct 20, 2018

@adam3smith @zuphilip i have a bunch of translators ready for review but buried in the queue—wanted to make sure you saw

@adam3smith
Copy link
Collaborator

@owcz thanks for the ping -- will try to get to soon-ish, but this week is still extraordinarily busy.

@zuphilip zuphilip removed the on hold label Oct 27, 2018
@zuphilip
Copy link
Contributor

Busy times for me as well...

@owcz
Copy link
Contributor Author

owcz commented Oct 27, 2018

no worries, just wanted to put it on your radar
all of these buried PRs in fact are ready for review

It's fine for catalog entry URLs to go in item.url, so long as they
provide some sort of full-text source for the item and would be useful
to include in a citation. That is the case here, so we'll include them.
@AbeJellinek
Copy link
Member

I'm sorry this sat in the backlog for so long! Great translator and a really cool site that I'm glad I know about now.

@AbeJellinek AbeJellinek merged commit d7d35bc into zotero:master Jul 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Translator Pull requests for new translators
Development

Successfully merging this pull request may close these issues.

4 participants