Skip to content

选择发送消息类型和发送消息后反馈消息格式转换bug#1

Open
winterbang wants to merge 5 commits intoraymondmars:masterfrom
winterbang:master
Open

选择发送消息类型和发送消息后反馈消息格式转换bug#1
winterbang wants to merge 5 commits intoraymondmars:masterfrom
winterbang:master

Conversation

@winterbang
Copy link

No description provided.

@raymondmars
Copy link
Owner

@winterbang 抱歉暂时不能 merge你的request,因为:
1,目前我还没有重现你提交的格式转换bug,能否进一步提交具体的单元测试样例?
2. 我用单列类来提供了 ios, android 的实例接口,为了提供通用性,不建议在 base 类的初始化方法中调用配置变量,具体调用样例参加我新加的单元测试。
3. 安卓的参数,目前只考虑了消息传递,没有考虑消息透传,如果要添加的更为通用一些,建议参数写成: (token, title, content, custom_content={}, params={}, msg_type = 1) 参数后置,并提供一个默认值,这样不会影响到之前的版本。
谢谢的Request .这次我完善了单元测试,你可以修改代码后,完成对应的单元测试。

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