Skip to content

修复appid提示不适用于所有对应适配器的问题#7746

Merged
Soulter merged 1 commit intoAstrBotDevs:masterfrom
idiotsj:fix/platform-appid-hint-7107
Apr 23, 2026
Merged

修复appid提示不适用于所有对应适配器的问题#7746
Soulter merged 1 commit intoAstrBotDevs:masterfrom
idiotsj:fix/platform-appid-hint-7107

Conversation

@idiotsj
Copy link
Copy Markdown
Contributor

@idiotsj idiotsj commented Apr 23, 2026

变更说明

  • 将平台配置中共享 �ppid 字段的提示文案改为平台中性描述
  • 同步更新后端配置元数据与前端中/英/俄文案,避免前后端说明不一致
  • 不修改任何适配器配置结构和运行逻辑,仅修正文案提示

验证情况

  • 未运行测试(仅文案/元数据修改)

Fixes #7107

Summary by Sourcery

Update configuration metadata hints to use platform-neutral AppID wording and align backend and frontend descriptions.

Enhancements:

  • Clarify the shared AppID field hint to be platform-neutral across chat providers.

Documentation:

  • Synchronize AppID configuration hint text across backend metadata and frontend i18n strings in Chinese, English, and Russian.

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 generalizes the appid configuration hint across the core configuration and multiple localization files (English, Russian, and Chinese), replacing platform-specific references to the QQ Official Bot with generic messaging platform terminology. A suggestion was made to improve the Russian translation to include instructions on how to obtain the App ID, ensuring it provides the same level of detail as the English and Chinese versions.

"appid": {
"description": "ID приложения",
"hint": "Обязательно для QQ Official Bot. См. документацию."
"hint": "Обязательно. App ID текущей платформы сообщений. См. документацию по интеграции платформы."
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The updated hint text is slightly less informative compared to the English and Chinese versions, as it omits the "how to obtain it" part. A more complete translation would be more helpful for Russian-speaking users.

Here's a suggested improvement to make it more explicit:

Suggested change
"hint": "Обязательно. App ID текущей платформы сообщений. См. документацию по интеграции платформы."
"hint": "Обязательно. App ID для текущей платформы. Как его получить, см. в документации по интеграции с платформой."

@idiotsj idiotsj changed the title fix(platform): clarify shared appid hint text 修复appid提示不适用于所有对应适配器的问题 Apr 23, 2026
@idiotsj idiotsj marked this pull request as ready for review April 23, 2026 10:19
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Apr 23, 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 reviewed your changes and they look great!


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.

@dosubot dosubot Bot added area:core The bug / feature is about astrbot's core, backend area:webui The bug / feature is about webui(dashboard) of astrbot. labels Apr 23, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Apr 23, 2026
@Soulter Soulter merged commit 489e2a3 into AstrBotDevs:master Apr 23, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:core The bug / feature is about astrbot's core, backend area:webui The bug / feature is about webui(dashboard) of astrbot. 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.

[Bug]错误提示

2 participants