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

又是我😂 #351

Merged
merged 6 commits into from Oct 21, 2018

Conversation

Projects
None yet
2 participants
@nondanee
Copy link
Contributor

commented Oct 20, 2018

对调用透明的

  • 重写了加密部分,使用自带库实现,去掉了对 big-integer 的依赖
  • 增加对 linux forward api 的支持
  • 整理了下 app.js(emmm按我自己的代码风格)

对调用有影响的

  • /check/music 已知 bug 修复
  • /top/song 增加 type 参数
  • /banner 换成 linux api,返回结构有所变动

按上次说的把 js cookie 去掉了,已知有问题的都单独 patch 了,可能还会有别的,等 issue 反馈吧

nondanee added some commits Oct 19, 2018

@Binaryify Binaryify merged commit 3bd2bcf into Binaryify:master Oct 21, 2018

@Binaryify

This comment has been minimized.

Copy link
Owner

commented Oct 21, 2018

才发现文档里没有 /top/song

@nondanee

This comment has been minimized.

Copy link
Contributor Author

commented Oct 21, 2018

emmm 没有废弃😓,Windows 版和 UWP 版都在用

Binaryify pushed a commit that referenced this pull request Oct 21, 2018

binaryify
@nondanee

This comment has been minimized.

Copy link
Contributor Author

commented Oct 23, 2018

尴尬了,我发现 maintainability 越改越低。。。😓

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.