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 Blouin Artinfo.js #1270

Closed
wants to merge 2 commits into from
Closed

Add Blouin Artinfo.js #1270

wants to merge 2 commits into from

Conversation

owcz
Copy link
Contributor

@owcz owcz commented Mar 14, 2017

Translator for http://www.blouinartinfo.com, an art news website. Handles all main news and blog pages, as well as all major category pages. The site is a mess and I couldn't get its search to work so no multi for that.

Everything runs fine with doWeb and browser, but I had some issues with Scaffold's testing environment (filed at zotero/scaffold/issues/38).

@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
  • now expands on EM
  • multi support for homepage, sections, search
  • this site is in terrible shape. i considered not even bothering with rewriting the translator. much of the old site, artinfo.com, appears inaccessible so not sure if thislower quality content will be worthwhile

@owcz
Copy link
Contributor Author

owcz commented Dec 31, 2021

Site is now defunct

@owcz owcz closed this Dec 31, 2021
@owcz owcz deleted the blouin-artinfo branch December 31, 2021 10:03
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 on hold
Development

Successfully merging this pull request may close these issues.

2 participants