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: Fix build error and /tmp/anonymous_token not exist. #2187

Merged
merged 3 commits into from
Mar 4, 2024

Conversation

DaiQiangReal
Copy link
Contributor

@DaiQiangReal DaiQiangReal commented Mar 4, 2024

  • 修复打包 node 侧代码时,因第三方依赖使用了新 ES 语法, webpack版本过低不识别的问题。(通过将 node_modules 过一遍 esbuild 来缓解,但如果继续维护旧播放器,建议升级 webpack)

  • 修复直接使用 NeteaseCloudMusicApi/server 未使用 /main.js 导致一些必要文件未被创建的问题。(另,NeteaseCloudMusicApi 的 generateConfig.js 也有问题,其从 main.js 引入了 register_anonimous,但 register_anonimous 并不存在,因此直接拷贝必要代码到项目中使用)


  • Fixed the issue where the webpack version was too low and not recognized due to the new ES syntax used in third-party dependencies. (Solved by passing node_modules through esbuild, but if maintenance continues, it is recommended to upgrade webpack)

  • Fixed an issue where using NeteaseCloudMusicApi/server directly without using /main.js resulted in some necessary files not being created. (In addition, NeteaseCloudMusicApi's generateConfig.js also has problems. It introduces register_anonimous from main.js, but register_anonimous does not exist, so copy the necessary code directly to the project for use)

Copy link

vercel bot commented Mar 4, 2024

@DaiQiangReal is attempting to deploy a commit to the r3play Team on Vercel.

A member of the Team first needs to authorize it.

@pan93412 pan93412 merged commit ae352f2 into qier222:master Mar 4, 2024
0 of 7 checks passed
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