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

Fix: get_forward_msg 接口返回类型注释错误 #88

Merged
merged 1 commit into from
Feb 4, 2024

Conversation

This-is-XiaoDeng
Copy link
Contributor

根据 OneBot V11 标准get_forward_msg 的响应数据应该是 dict[str, Any],而不是 None

OneBot V11 标准原文

get_forward_msg 获取合并转发消息

参数

字段名 数据类型 说明
id string 合并转发 ID

响应数据

字段名 类型 说明
message message 消息内容,使用 消息的数组格式 表示,数组中的消息段全部为 node 消息段

Copy link

github-actions bot commented Feb 4, 2024

@github-actions github-actions bot temporarily deployed to pull request February 4, 2024 04:29 Inactive
@yanyongyu yanyongyu changed the title 🐛 Fix: get_forward_msg 接口返回类型注释错误 Fix: get_forward_msg 接口返回类型注释错误 Feb 4, 2024
Copy link

codecov bot commented Feb 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (51f114b) 67.70% compared to head (b94309f) 67.70%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #88   +/-   ##
=======================================
  Coverage   67.70%   67.70%           
=======================================
  Files          23       23           
  Lines        2081     2081           
=======================================
  Hits         1409     1409           
  Misses        672      672           
Flag Coverage Δ
unittests 67.70% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@yanyongyu yanyongyu added the bug Something isn't working label Feb 4, 2024
@yanyongyu yanyongyu merged commit f3d9c51 into nonebot:master Feb 4, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

Successfully merging this pull request may close these issues.

None yet

2 participants