Skip to content

fix: wrong profile update url #105

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
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix: wrong profile update url #105

wants to merge 2 commits into from

Conversation

liuly0322
Copy link
Collaborator

目前后端还有问题仍待解决

  • 密码的更新需要采用 make_password 方法，因为密码并不直接存储明文
  • 当前 User 没有介绍，学校，专业，班级，真实姓名，qq 这些字段
  • 学号 student_id 应该设置不允许更新

daiwenyuan and others added 2 commits January 3, 2023 19:10
使用docker-compose 进行 DEV模式运行
44e22c9 
注意：当使用PRODUCTION模式运行时，需要配置 ALLOWED_HOST，添加正式的域名
@liuly0322
fix: wrong profile update url
edfd8cf
@libreliu
Copy link
Collaborator

libreliu commented Feb 7, 2023

目前的思路：后端把 UserViewSet 拆开，GET 还用原来的 Mixin，PUT 改成一个专门的处理用户更新的 View 来做

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@liuly0322 @libreliu