Abe Jellinek

Results 189 comments of Abe Jellinek

Also see my earlier comments, which shouldn't have been resolved.

Strip the tags, but otherwise, looks good!

Thanks, and sorry for the delay in reviewing this? Generally everything looks good, but it would be best to convert this to an async translator. See [CourtListener](https://github.com/zotero/translators/blob/master/CourtListener.js) for an example....

> I'm not able to build this with Node 21 I just undid a config change that made it in by mistake. Otherwise there's nothing in here that should affect...

Thanks, looks good, just one nitpick.

Oh, and some naming nitpicks: - It looks like the site calls itself LIBRIS, not Libris. - We should follow our naming convention for other ISBN translators and call this...

Just setting `item.DOI` no matter whether the item type claims to support it should work: ```js if (zfield == 'DOI' || ZU.fieldIsValidForType(zfield, newItem.itemType)) { ```

`pub-location` is CSL `publisher-place` ("Geographic location of the publisher")? If so, setting `item.place` or `Place:` in `extra` should work.