Page MenuHomePhabricator

Add new translators
Closed, ResolvedPublic1 Estimated Story Points

Description

Please sync the following translators from https://github.com/zotero/translators

  • Eurogamer.js
  • IGN.js
  • USgamer.js

If there's a better way to request a pull, let me know, as I'm just following as directed

Event Timeline

Restricted Application added subscribers: Zppix, Aklapper. · View Herald Transcript
czar removed Mvolz as the assignee of this task.Jun 16 2016, 8:34 AM
czar updated the task description. (Show Details)
czar added a subscriber: Mvolz.
Mvolz triaged this task as Medium priority.Jul 29 2016, 2:52 PM
Mvolz moved this task from Backlog to Service on the Citoid board.
Mvolz added a project: Services.
Mvolz added a subscriber: mobrovac.

zotero/translators recently pulled a modified version of @Mvolz 's PR (1100). If I'm not mistaken, I think those were the last issues holding up full automated sync with the translator repo?

@czar this is the last hold-up: T113262

have to decide if we need to strip the search query... if we don't then we can start doing automated sync. But there are some mild privacy objections... we could strip the search string in citoid instead of a zotero translators though perhaps?

Do the translators even have access to the input url?

@Mvolz yep, the translator can modify the active/input URL for the citation's "url" parameter

Mvolz reassigned this task from Mvolz to mobrovac.