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

修改 app/learn 的 course/challenge #144

Merged
merged 3 commits into from
Feb 19, 2025
Merged

修改 app/learn 的 course/challenge #144

merged 3 commits into from
Feb 19, 2025

Conversation

nmsn
Copy link
Contributor

@nmsn nmsn commented Feb 16, 2025

处理 #128

  • 使用 useMounted 方法替代无依赖的 useEffect
  • helper 文件移入 domain/course
  • 使用映射替代重复判断

course/challenge 很多组件是共用的,使用新目录转移相关代码的功能还没有理解的很清晰,暂时没改,感觉还需要指导一下

Copy link

vercel bot commented Feb 16, 2025

@nmsn is attempting to deploy a commit to the OpenBuild Team on Vercel.

A member of the Team first needs to authorize it.

@ourai ourai merged commit 8b0e8c4 into openbuildxyz:test Feb 19, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

用户侧课程与活动相关代码适配新目录结构并采用新的数据请求方式
2 participants