windingwind / zotero-pdf-translate

Translate PDF, EPub, webpage, metadata, annotations, notes to the target language. Support 20+ translate services.
GNU Affero General Public License v3.0
6.15k stars 304 forks source link

add huoshan translate service #780

Open windwhim opened 1 month ago

windwhim commented 1 month ago

add huoshan translate service

windingwind commented 3 weeks ago

整体ok,两个小建议/问题:

  1. 建议尽可能减少依赖第三方包(如crypto-js),使用browser自带的crypto对象,见utils/crypto.ts,其中有一些已有的方法。如果这部分有困难,先按照目前合并也可。
  2. crypto-js不在目前的package.json,如果依赖,需要添上。
windwhim commented 3 weeks ago

整体ok,两个小建议/问题:

  1. 建议尽可能减少依赖第三方包(如crypto-js),使用browser自带的crypto对象,见utils/crypto.ts,其中有一些已有的方法。如果这部分有困难,先按照目前合并也可。
  2. crypto-js不在目前的package.json,如果依赖,需要添上。

在之后我尝试使用crypto.ts重写

windwhim commented 2 weeks ago

整体ok,两个小建议/问题:

  1. 建议尽可能减少依赖第三方包(如crypto-js),使用browser自带的crypto对象,见utils/crypto.ts,其中有一些已有的方法。如果这部分有困难,先按照目前合并也可。
  2. crypto-js不在目前的package.json,如果依赖,需要添上。

在使用crypto.ts时,它的hmacSha256Digest返回的为一个Promise,而crypto-js中HmacSHA256返回的是一个object,我不知道要怎样解决,所以暂时在package.json添加了crypto-js依赖