search

nonebot / adapter-qq

NoneBot2 QQ 适配器 / QQ adapter for nonebot2
MIT License
205 stars 18 forks source link

:pencil2: remove author id #86

Closed Dreamail closed 9 months ago

Dreamail commented 9 months ago

it no use and case some bug in saa

yanyongyu commented 9 months ago

目前不考虑移除，只有QQ确认移除后才会删除

Dreamail commented 9 months ago

没确认？ 旧文档只有频道API吧，况且频道API用的model是另一个 新文档里c2c和group都没有id字段

yanyongyu commented 9 months ago

目前为保证兼容性该字段未移除，仅在新文档内隐藏，旧文档里依旧保留

Dreamail commented 9 months ago

旧文档已经不适用了啊 ws接口都用新文档的定义了吧

（况且adapter-qq更多是用来接group c2c的吧，旧的频道adapter不是有qqguild？

Dreamail commented 9 months ago

保留这个字段并且不设optional会导致校验错误吧

yanyongyu commented 9 months ago

现在群机器人是两种字段同时推送以保证兼容性，什么时候移除了适配器再跟进

Dreamail commented 9 months ago

问题是群和c2c已经移除了啊（

yanyongyu commented 9 months ago

事实上这两个字段都存在，你可以自己去试一下

Dreamail commented 9 months ago

还真是（和openid一样 没事了orz