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

清理一下 User 类下的方法 #436

Open
weakish opened this issue Dec 13, 2019 · 0 comments
Open

清理一下 User 类下的方法 #436

weakish opened this issue Dec 13, 2019 · 0 comments
Labels
next breaking changes

Comments

@weakish
Copy link
Contributor

weakish commented Dec 13, 2019

比如:

  1. 大量的 request_xxx class method 没有使用 class、instance 信息,直接发一个请求,其实是 static method。这些方法中,部分应该改成 static method,部分应该改成实例方法,并更新实例的相应字段,比如 emailVerified 之类。
  2. login 是实例方法,login_with_mobile_phone 却是 class method,这样的不一致性并不必要。可以废弃 login_with_mobile_phone,并入 login 方法(增加一个 mobile 参数)。
@weakish weakish added the next breaking changes label Dec 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next breaking changes
Projects
None yet
Development

No branches or pull requests

1 participant