Skip to content

Conversation

@juvenn
Copy link
Member

@juvenn juvenn commented Jun 24, 2019

No description provided.

@juvenn juvenn requested a review from weakish June 24, 2019 10:02
Copy link
Contributor

@weakish weakish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

看了一下其他 SDK 的行为,js 和 java 都会检查手机号、验证码是否为空, 不过这个不查也没关系,留给云端报错也可以。

另外,js 是返回一个结果(成功、失败),java 是回调传入 exception 和 user(但是 user 并不会从云端去同步字段)。

不过 php sdk 的做法倒是和现在 python sdk 是一致的(不检查参数,同步用户信息)。

@juvenn juvenn merged commit 1d8a592 into leancloud:master Jun 24, 2019
@juvenn juvenn deleted the feat/signup-sms branch June 24, 2019 13:46
@juvenn
Copy link
Member Author

juvenn commented Jun 24, 2019

准备先发个版,修复一下内心的尴尬😅再说。后面还要麻烦 @weakish 帮忙梳理细节问题。

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