search

zotero / zotero-connectors

Chrome, Firefox, Edge, and Safari extensions for Zotero
https://www.zotero.org/download/connectors
Other
480 stars 119 forks source link

`Zotero.HTTP.request()`: Support 'json' responseType, return URL for `wrapDocument()` #408

Closed AbeJellinek closed 1 year ago

AbeJellinek commented 1 year ago

So apparently request() in the Connector didn't support returning JSON, which has been breaking our very few translators that call requestJSON(). Go figure! I probably should've caught that before merging in the async request functions. We also weren't returning responseURL, which requestDocument() uses to construct a wrapped Document.

(Some of these may be MV3 regressions, not sure, but I'm seeing them in the latest stable version on Firefox.)

dstillman commented 1 year ago

Pushed to 5.0.102beta1 for Firefox

AbeJellinek commented 1 year ago

Let me know when 5.0.102 is released so I can push the translator update.