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

feat:【企业微信】新增待开发应用获取带参授权链接接口 #2846

Merged
merged 2 commits into from
Oct 21, 2022

Conversation

freedom9
Copy link

@binarywang
Copy link
Owner

感谢楼主提供PR,不过为了保证代码质量,建议加上单元测试代码,如果确实不方便编写,或无法测试,请给出自测成功的截图也可以

@sonatype-lift
Copy link

sonatype-lift bot commented Oct 21, 2022

⚠️ 22 God Classes were detected by Lift in this project. Visit the Lift web console for more details.

@freedom9
Copy link
Author

image
由于一些参数不方便给出,只能给出成功截图。

@binarywang
Copy link
Owner

其实你是写了单元测试的吧,建议将敏感账号信息参数抽取成配置项,这样就可以把单元测试代码提交上来,方便其他人以后做测试验证

@binarywang binarywang merged commit 9b78acd into binarywang:develop Oct 21, 2022
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