Skip to content

docs: corrent weixin_oc wechat version requirements#7068

Merged
Soulter merged 2 commits intoAstrBotDevs:masterfrom
tlw00988:patch-1
Mar 28, 2026
Merged

docs: corrent weixin_oc wechat version requirements#7068
Soulter merged 2 commits intoAstrBotDevs:masterfrom
tlw00988:patch-1

Conversation

@tlw00988
Copy link
Copy Markdown
Contributor

@tlw00988 tlw00988 commented Mar 28, 2026

Modifications / 改动点

经过实践检测,在Android微信8.0.69下,也有ClawBot插件可用,而无需更新到微信8.0.70(事实上Android也没有这个版本号)

  • This is NOT a breaking change. / 这不是一个破坏性变更。

Screenshots or Test Results / 运行截图或测试结果

图片 图片

Checklist / 检查清单

  • 😊 If there are new features added in the PR, I have discussed it with the authors through issues/emails, etc.
    / 如果 PR 中有新加入的功能,已经通过 Issue / 邮件等方式和作者讨论过。
    (我压根没加功能哈哈)

  • 👀 My changes have been well-tested, and "Verification Steps" and "Screenshots" have been provided above.
    / 我的更改经过了良好的测试,并已在上方提供了“验证步骤”和“运行截图”

  • 🤓 I have ensured that no new dependencies are introduced, OR if new dependencies are introduced, they have been added to the appropriate locations in requirements.txt and pyproject.toml.
    / 我确保没有引入新依赖库,或者引入了新依赖库的同时将其添加到 requirements.txtpyproject.toml 文件相应位置。

  • 😮 My changes do not introduce malicious code.
    / 我的更改没有引入恶意代码。

Summary by Sourcery

Documentation:

  • Clarify Weixin OC adapter documentation with separate minimum WeChat versions for iOS (>= 8.0.70) and Android (>= 8.0.69) and note the need for the ClawBot plugin.

Update WeChat version requirements for the adapter.
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Mar 28, 2026
Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - I've left some high level feedback:

  • The updated requirement string mixes iOS and Android in a single sentence; consider splitting into two bullet points or lines (one for iOS, one for Android) to improve readability and avoid confusion.
  • The phrase 确保插件内有插件微信ClawBot is a bit ambiguous; you might clarify whether users should verify the ClawBot plugin is installed, enabled, or visible in a specific location in the UI.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- The updated requirement string mixes iOS and Android in a single sentence; consider splitting into two bullet points or lines (one for iOS, one for Android) to improve readability and avoid confusion.
- The phrase `确保插件内有插件微信ClawBot` is a bit ambiguous; you might clarify whether users should verify the ClawBot plugin is installed, enabled, or visible in a specific location in the UI.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the WeChat adapter documentation to specify minimum version requirements for iOS (8.0.70) and Android (8.0.69) and adds a note regarding the ClawBot plugin. Feedback was provided to improve the phrasing and formatting of this note for better clarity and consistency.

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 28, 2026
@Soulter Soulter changed the title docs:corrent weixin_oc wechat version requirements docs: corrent weixin_oc wechat version requirements Mar 28, 2026
@Soulter Soulter merged commit 9238ad5 into AstrBotDevs:master Mar 28, 2026
1 check passed
@tlw00988 tlw00988 deleted the patch-1 branch March 28, 2026 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants