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.

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

required必填项没有校验 #28

Closed
bill1314 opened this issue Aug 6, 2018 · 2 comments
Closed

required必填项没有校验 #28

bill1314 opened this issue Aug 6, 2018 · 2 comments

Comments

@bill1314
Copy link

bill1314 commented Aug 6, 2018

作者辛苦了，required必填项没有校验
@caspar-chen
Copy link
Owner

谢谢反馈，这个校验，原先设计的时候是没打算加的，因为需要测试前端不传必填参数的时候，后端返回情况。是否做了正确的异常处理等。
关于这个问题也可讨论是否有必要加。

@caspar-chen
Copy link
Owner

经过考虑，不进行必填校验

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

No branches or pull requests
2 participants
@caspar-chen @bill1314