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.

Sign up for GitHub

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

Jump to bottom

fix: Update dependencies and fix some CVE. #34

Merged

Jisu-Woniu merged 2 commits into NJUPT-SAST:main from Jisu-Woniu:main

Jan 25, 2024

Member

Jisu-Woniu commented Jan 20, 2024 •

edited

Loading

还有一个 Cypress 的 CVE，但是看项目里面没有什么有意义的测试文件


          fix: Update dependencies and fix some CVE.

5fc1da2

Jisu-Woniu requested a review from love98ooo

January 20, 2024 09:37

Member Author

Jisu-Woniu commented Jan 20, 2024

另外，路由的不同页面似乎应该用 import("page") 动态加载，防止首页打开的时候加载的 JS 文件过大


          Ignore cypress testing output.

0f61e5f

Jisu-Woniu merged commit 5067e41 into NJUPT-SAST:main

2 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet