Page MenuHomePhabricator
Create Task
Maniphest T211114

Some regressions in production with Zotero translation-server in production
Closed, ResolvedPublic
Actions

Description

These all should have come back with 'zotero' in the source:

https://en.wikipedia.org/api/rest_v1/data/citation/mediawiki/Q33415777

Related Objects
Search...

Event Timeline

Mvolz triaged this task as Unbreak Now! priority.Dec 4 2018, 2:25 PM
Mvolz created this task.
Restricted Application added subscribers: Liuxinyu970226, TerraCodes, Aklapper. · View Herald TranscriptDec 4 2018, 2:25 PM
Mvolz updated the task description. (Show Details)Dec 4 2018, 2:26 PM
Mvolz mentioned this in T197242: Transition citoid to use Zotero's translation-server-v2.Dec 4 2018, 2:32 PM
Jdforrester-WMF added a comment.Dec 4 2018, 3:38 PM

Looks fixed now?

Though the date format returned is not ideal (see https://en.wikipedia.org/api/rest_v1/data/citation/mediawiki/https%3A%2F%2Fwww.bbc.com%2Fnews%2Fworld-europe-46437904 - "accessDate":"2018-12-04T15:38:08Z" is different from before, I think?

Mvolz added a comment.EditedDec 4 2018, 3:46 PM
This comment has been deleted.
Mvolz closed this task as Resolved.Dec 4 2018, 3:46 PM

Hmm but the qid one is still not working...

Mvolz renamed this task from Citoid not communicating with Zotero translation-server in production at all to Some regressions in production with Zotero translation-server in production at all.Dec 4 2018, 3:50 PM
Mvolz reopened this task as Open.
Mvolz lowered the priority of this task from Unbreak Now! to High.
Mvolz lowered the priority of this task from High to Medium.
Mvolz updated the task description. (Show Details)
Mvolz updated the task description. (Show Details)
Mvolz added a comment.Dec 4 2018, 4:32 PM
In T211114#4797913, @Jdforrester-WMF wrote:

Looks fixed now?

Though the date format returned is not ideal (see https://en.wikipedia.org/api/rest_v1/data/citation/mediawiki/https%3A%2F%2Fwww.bbc.com%2Fnews%2Fworld-europe-46437904 - "accessDate":"2018-12-04T15:38:08Z" is different from before, I think?

Good catch; I think the access date is something we just didn't test for, that needs to be patched. Putting the access date through is a new translation-server feature but we'll have to strip the time info.

• mobrovac closed this task as Resolved.Dec 4 2018, 4:56 PM
• mobrovac assigned this task to akosiaris.

There was a communication problem both between Citoid and Zotero as well as Zotero and our domains. This has been fixed now.

Mvolz added a comment.Dec 4 2018, 8:12 PM

The qid issue is still ongoing; I suspect that Zotero is crashing with that
request.

• mobrovac closed subtask T211127: The new translation-server returns access date with the full time stamp; we should strip this as Resolved.Dec 5 2018, 11:32 AM
Liuxinyu970226 unsubscribed.Dec 9 2018, 7:15 AM
Mvolz closed subtask T211148: QIDs work locally but not in production with new translation-server as Resolved.Dec 9 2018, 10:05 AM
Mvolz renamed this task from Some regressions in production with Zotero translation-server in production at all to Some regressions in production with Zotero translation-server in production.Dec 20 2018, 2:22 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL