Maniphest T370685

Make Zotero return more specific errors codes
Open, Needs TriagePublic
Actions

Assigned To

None

Authored By

	ppelberg
	Jul 22 2024, 5:39 PM

Tags

Referenced Files

None

Subscribers

Description

This task involves the work of making Zotero return more specific error codes so that we can offer people a generic feedback message when they attempt to cite a source Citoid doesn't support (T369547)

Sequence

Make changes in Zotero so that it returns more specific error codes to Citoid
- Upstream implementation: https://github.com/zotero/translation-server/pull/171#issuecomment-2271274898
- Local implementation: https://gerrit.wikimedia.org/r/c/mediawiki/services/zotero/+/1054354
In Citoid, we need to pass on error message "1." will implement on to the front-end
Implement front-end changes:
- T369547

Open questions

What will we do in the time between now and when Zotero will upstream these changes?

Details

Due Date: Aug 21 2024, 3:00 PM

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Open		None	T362379 Several major news websites (NYT, NPR, Reuters...) block citoid
		Open		Mvolz	T364594 Revise Citoid error message to be more specific
		Open		dchan	T369547 Introduce a generic unsupported format error message within Citoid UI
		Open	Feature	None	T370263 Review the entire flow of interaction between VisualEditor Citoid extension, the Citoid service and Zotero
		Open		None	T370685 Make Zotero return more specific errors codes

Event Timeline

ppelberg created this task.Jul 22 2024, 5:39 PM

ppelberg added a parent task: T369547: Introduce a generic unsupported format error message within Citoid UI.

Citoid service patch https://gerrit.wikimedia.org/r/c/mediawiki/services/zotero/+/1054354

Same patch, but upstreamed to Zotero: https://github.com/zotero/translation-server/pull/171

ppelberg updated the task description. (Show Details)Jul 23 2024, 7:04 PM

We're waiting for Zotero maintainer(s) to review patch upstream.

Proposed path forward
If we don't hear back from Zotero by 21 August 2024 [i] we'll merge the local patch.

Note: moving forward with the above needs sign-off from Editing Engineering.

i. This date is 2 weeks after @Mvolz commented on the pull request @zoe will have made 5 weeks ago from this future point.

ppelberg set Due Date to Aug 21 2024, 3:00 PM.Aug 8 2024, 9:23 PM

ppelberg updated the task description. (Show Details)Aug 13 2024, 5:15 PM

Per today's Editing Team planning meeting, we're going to merge the local patch now.