support additional api for llonebot #113
Conversation
Qodana Community for JVMIt seems all right 👌 No new problems were found according to the checks applied 💡 Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at qodana-support@jetbrains.com
|
|
首先感谢贡献，不过🤔我个人认为作为针对标准API的额外API扩展，它更适合以第三方扩展库的形式，不知是否有意向考虑将这提供的 llonebot 的 API （以及gocqhttp的API）以一种第三方的库发布呢？ |
| @@ -123,6 +125,8 @@ public class OneBotForwardNode private constructor( | |||
| @SerialName("user_id") | |||
| val userId: ID? = null, | |||
| val nickname: String? = null, | |||
| val name: String? = null, | |||
| val uin: ID? = null, | |||
There was a problem hiding this comment.
针对这俩额外添加的属性提供一些文档注释说明（比如它们是哪些平台所需要的）
There was a problem hiding this comment.
这是llonebot实现的cqhttp合并转发消息的两个属性，name和uin nickname和userId，意思是一样的，只是名字不同
There was a problem hiding this comment.
这是llonebot实现的cqhttp合并转发消息的两个属性，name和uin nickname和userId，意思是一样的，只是名字不同
那么直接将它们作为普通属性而不是构造参数加在类中即可，并对应的 nickname 和 userId 初始化，这样可以确保 Data 的二进制兼容性，给这俩属性加个文档注释简单说一下它们是 llonebot 中的属性就好
api的请求和返回数据结构都是一致的，并且也需要继承其OneBotApi，是需要第三方库引入带有OneBotApi的依赖吗？ |
嗯是这个意思。作为第三方库，引入组件库作为依赖并实现 |
支持llonebot的非onebot标准的api。