-
Notifications
You must be signed in to change notification settings - Fork 406
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Volcano Translate #636
Volcano Translate #636
Conversation
我先准备一下发布新版本,稍后再看。 |
我想了想,Volcano 这里我们还是不要添加默认的 key 了,直接让用户去自己申请 key 更好,如果没有填写,直接报错提示。 目前 Easydict 支持的免费翻译服务已经比较多了,一般有 DeepL, Google, Bing 这些够用了。 额外维护这些 key 比较麻烦,如果使用我们自己的 key,会占有自己的翻译额度,甚至被超支收费。 像目前腾讯翻译的 key,是我使用邮箱注册的小号申请的,但不是所有服务都支持小号,例如火山翻译就要求手机号,因此这种我们就不要内置 key,让用户自己去折腾好了。 |
对于这种比较麻烦 API 签名问题,如果没有官方 SDK 或者 demo,我们可以直接去 GitHub 找现成的代码示例,例如直接搜代码 https://github.com/search?q=Action%3DTranslateText%26Version%3D+language%3ASwift+&type=code 我没测过上面代码,你自己跑一下看看。 ![]() |
哈哈哈未曾设想的道路 😂 这两天有点事,有空了就看一下 |
现在就只有最后一个问题 在用户第一次输入 API key 后立刻点验证,会发现报错,我是在 |
你的写法不对,这里不应使用存储属性,应该使用计算属性 5a01ba0 |
原来如此,感谢 |
I canceled some of the workflows to save up CI minute |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
Closes #255