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

Adapter: RocketChat #2636

Closed
IllTamer opened this issue Apr 10, 2024 · 9 comments · Fixed by #2637
Closed

Adapter: RocketChat #2636

IllTamer opened this issue Apr 10, 2024 · 9 comments · Fixed by #2637
Labels
Adapter Publish Adapter

Comments

@IllTamer
Copy link
Contributor

适配器名称

RocketChat

适配器描述

RocketChat adapter for nonebot2

PyPI 项目名

nonebot-adapter-rocketchat

适配器 import 包名

nonebot.adapters.rocketchat

适配器项目仓库/主页链接

https://github.com/IUnlimit/nonebot-adapter-rocketchat

标签

[]

@IllTamer IllTamer added the Adapter Publish Adapter label Apr 10, 2024
noneflow bot pushed a commit that referenced this issue Apr 10, 2024
@noneflow
Copy link
Contributor

noneflow bot commented Apr 10, 2024

📃 商店发布检查结果

Adapter: RocketChat

✅ 所有测试通过,一切准备就绪!

详情
  • ✅ 项目 nonebot-adapter-rocketchat 已发布至 PyPI。
  • ✅ 项目 主页 返回状态码 200。

  • 💡 如需修改信息,请直接修改 issue,机器人会自动更新检查结果。
    💡 当插件加载测试失败时,请发布新版本后在当前页面下评论任意内容以触发测试。

    ♻️ 评论已更新至最新检查结果

    💪 Powered by NoneFlow

    @yanyongyu
    Copy link
    Member

    yanyongyu commented Apr 10, 2024

    @IllTamer 另外请在nb仓库README支持的协议部分添加一下相关信息,可以参考已有内容的格式,单独开一个新的PR

    @RF-Tar-Railt
    Copy link
    Member

    #2435 你可以阅读参考一下这个

    以及请问后续你有打算完善消息段吗,我看你目前只写了text但是注释里已经有emoji,mention等元素了

    @IllTamer
    Copy link
    Contributor Author

    @IllTamer 另外请在nb仓库README支持的协议部分添加一下相关信息,可以参考已有内容的格式,单独开一个新的PR

    好的,感谢您的审查,我会在规范代码后更新 README 并重新 pr

    @yanyongyu
    Copy link
    Member

    好的,感谢您的审查,我会在规范代码后更新 README 并重新 pr

    不需要重新提交 issue,更新 README 直接新开一个 pr 即可

    @IllTamer
    Copy link
    Contributor Author

    #2435 你可以阅读参考一下这个

    以及请问后续你有打算完善消息段吗,我看你目前只写了text但是注释里已经有emoji,mention等元素了

    感谢您的分享。

    我后续会陆续完善其他监听事件的转换。事实上,Rocket.Chat 的消息发送格式较为特殊:对于支持文本内解析的消息段(如 emoji,@ 类型消息等),我会在后续更新中增加对应消息段的构建方法;对于期望发送其他类型消息的开发者,他们应该通过调用对应平台 API 的方式来实现,而非寄希望于适配器支持解析类似 onebot 协议的消息段。

    以上是我的观点,如果您有任何好的想法,我也很愿意进一步交流^ω^

    @RF-Tar-Railt
    Copy link
    Member

    #2435 你可以阅读参考一下这个

    以及请问后续你有打算完善消息段吗,我看你目前只写了text但是注释里已经有emoji,mention等元素了

    感谢您的分享。

    我后续会陆续完善其他监听事件的转换。事实上,Rocket.Chat 的消息发送格式较为特殊:对于支持文本内解析的消息段(如 emoji,@ 类型消息等),我会在后续更新中增加对应消息段的构建方法;对于期望发送其他类型消息的开发者,他们应该通过调用对应平台 API 的方式来实现,而非寄希望于适配器支持解析类似 onebot 协议的消息段。

    以上是我的观点,如果您有任何好的想法,我也很愿意进一步交流^ω^

    能够发送基础的那些消息内容(文字,emoji,@,图片)就好了,并不是强制要求所有类型消息都要塞在MessageSegment

    @noneflow noneflow bot closed this as completed in #2637 Apr 11, 2024
    noneflow bot pushed a commit that referenced this issue Apr 11, 2024
    @yanyongyu
    Copy link
    Member

    @IllTamer 请在NB仓库README里添加一下适配器信息,开PR即可

    @yanyongyu
    Copy link
    Member

    @IllTamer 适配器在启动的时候应该使用self.on_ready来处理setup的情况

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    Adapter Publish Adapter
    Development

    Successfully merging a pull request may close this issue.

    3 participants