leancloud / php-sdk

LeanCloud PHP SDK
Apache License 2.0
53 stars 23 forks source link

feat: 支持消息通讯API #184

Closed alberthuang24 closed 5 years ago

alberthuang24 commented 5 years ago

背景

公司有场景用到消息通讯, 结果发现php的sdk不支持. 只能自己写curl. 后面发现稍微改改sdk就可以支持了

alberthuang24 commented 5 years ago

...没有改旧的逻辑, 为什么测试用例会不过

juvenn commented 5 years ago

你好 @HongjiangHuang ,非常感谢你的 PR 。但是该 PR 中增加的接口与我们的 javascript sdk 很不一致。

如果你能够重新设计一下,新建一个继承自 LeanObject 的 Conversation 的类,并参考上面 javascript 的接口来添加相关的方法,那样会更棒。

alberthuang24 commented 5 years ago

不太明白, 为什么 PHP SDK 要跟 js SDK 的API保持一致?

juvenn commented 5 years ago

接入 LeanCloud 服务的应用通常会需要接入多个平台,我们希望尽量减少开发者在接入不同平台时的不一致性。

alberthuang24 commented 5 years ago

好的, 那我改改

juvenn commented 5 years ago

好的,辛苦了