Skip to content
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

fix(api): 修复新版 api 没有单词详细释义问题 #52

Merged
merged 5 commits into from
Feb 5, 2020

Conversation

zthxxx
Copy link
Contributor

@zthxxx zthxxx commented Feb 4, 2020

最近几次优化如下

  • 使用 Alfred 自带 variables 功能配置有道 API Keys (0c68ad3)
  • 增加发音快捷键、优化单词复制过滤正则 (0be6ed9)
  • 修复新版 Api 没有单词详细释义问题 (2cbeb67)
  • 更新 README,修复文案、按键、增加下载命令 (bb767b3)

API 问题造成只有一个单词翻译,没有其他列表信息,修复如下,

Before:

image

After fixed:

image


现在的 workflow 结构预览

image

close #47

@zthxxx zthxxx requested a review from wensonsmith February 4, 2020 08:55
@zthxxx
Copy link
Contributor Author

zthxxx commented Feb 4, 2020

@wensonsmith merge 之后还需要麻烦作者大大手动发一下 release 文件,

因为README 里面有写 release 的下载方式

@wensonsmith wensonsmith merged commit b84a5fb into wensonsmith:master Feb 5, 2020
@wensonsmith
Copy link
Owner

你真棒! 👍🏻

@zthxxx
Copy link
Contributor Author

zthxxx commented Feb 5, 2020

@wensonsmithProject 中的这个「文档帮助」 task 内的都已经改完了

image

@wensonsmith
Copy link
Owner

Release 已发
相关 Issue 已关闭

想法太多,业余项目太多,顾不过来了,你这个 PR 十分给力

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

下载安装后直接使用出错。
2 participants