Skip to content

docs(officecli): update assistant demo GIFs (PPT/Word/Excel)#1919

Merged
piorpua merged 3 commits intomainfrom
officecli-readme-gifs
Mar 30, 2026
Merged

docs(officecli): update assistant demo GIFs (PPT/Word/Excel)#1919
piorpua merged 3 commits intomainfrom
officecli-readme-gifs

Conversation

@ringringlin
Copy link
Copy Markdown
Collaborator

Summary

Update the README sections for OfficeCLI-powered PPT/Word/Excel assistants with new/optimized GIF assets.

Changes

  • readme.md: updated Office assistants (PPT/Word/Excel) section
  • docs/readme/readme_*.md: synced Excel/office assistant visuals across locales
  • resources/: added readme-demo-*.gif and updated morph-ppt-balanced.gif (smaller README-friendly size)

Related Issue

Closes #1918

Test Plan

  • Verify GIFs load in readme.md
  • Verify GIFs load in docs/readme/readme_*.md

Made with Cursor

@piorpua piorpua added the bot:reviewing Review in progress (mutex) label Mar 30, 2026
@piorpua
Copy link
Copy Markdown
Contributor

piorpua commented Mar 30, 2026

Code Review:docs(officecli): update assistant demo GIFs (PPT/Word/Excel) (#1919)

变更概述

本 PR 为 readme.md 和全部 locale README 文件(7 个语言版本)新增了 OfficeCLI 办公助手(PPT / Word / Excel)章节,并引入了 5 个新 GIF 演示资源。同时将所有 locale 文件中的旧版 Excel 演示图(generate_xlsx.gif)统一替换为新命名的 readme-demo-generate-excel.gif


方案评估

结论⚠️ 方案有缺陷

文档结构和内容组织合理,新增的两栏 <table> 布局与项目现有 README 风格一致;locale 文件同步更新也符合维护一致性的要求。但存在一处明确的资产遗漏(见下方问题清单),导致 WeChat 社区链接破损。


问题清单

🟠 HIGH — WeChat 群链接图片文件缺失

文件readme.md,第 37 行 & 第 607 行

问题代码

<!-- 第 37 行 -->
<a href="./resources/wechat_group_5.png" target="_blank">微信 (中文群)</a>

<!-- 第 607 行 -->
[WeChat Group](./resources/wechat_group_5.png)

问题说明:PR 将原来的 ./resources/wx-5.png 改为 ./resources/wechat_group_5.png,但仓库中并不存在 wechat_group_5.png 文件(resources/ 目录下只有 wx-4.pngwx-5.png)。合并后这两处链接将返回 404,用户无法扫码加入微信群。原文件 wx-5.png 仍在仓库中,仅需恢复引用或补充添加改名后的文件。

修复建议(二选一):

  1. 恢复原引用:
<!-- 第 37 行 -->
<a href="./resources/wx-5.png" target="_blank">微信 (中文群)</a>

<!-- 第 607 行 -->
[WeChat Group](./resources/wx-5.png)
  1. 或将 resources/wx-5.png 重命名/复制为 resources/wechat_group_5.png 并一并提交。

汇总

# 严重级别 文件 问题
1 🟠 HIGH readme.md:37, 607 WeChat 群链接图片文件缺失(wechat_group_5.png 不存在)

结论

⚠️ 有条件批准 — 存在一处 HIGH 级问题(WeChat 社区链接破损),修复后可合并。


本报告由本地 pr-review skill 生成,包含完整项目上下文,无截断限制。

@piorpua piorpua added bot:ready-to-fix CONDITIONAL review done, waiting for bot fix bot:fixing Fix in progress (mutex) and removed bot:reviewing Review in progress (mutex) bot:ready-to-fix CONDITIONAL review done, waiting for bot fix labels Mar 30, 2026
- Fix broken WeChat community link at line 37 (community bar)
- Fix broken WeChat community link at line 607 (community section)

Review follow-up for #1919
@piorpua
Copy link
Copy Markdown
Contributor

piorpua commented Mar 30, 2026

PR Fix 验证报告

原始 PR: #1919
修复方式: 直接推送到 officecli-readme-gifs

# 严重级别 文件 问题 修复方式 状态
1 🟠 HIGH readme.md:37, 607 WeChat 群链接图片文件缺失(wechat_group_5.png 不存在) 将两处 wechat_group_5.png 恢复为 wx-5.png ✅ 已修复

总结: ✅ 已修复 1 个 | ❌ 未能修复 0 个

@piorpua piorpua added bot:ready-to-merge Bot done, code is clean — human just needs to confirm and merge and removed bot:fixing Fix in progress (mutex) labels Mar 30, 2026
@piorpua piorpua merged commit cf4a013 into main Mar 30, 2026
21 checks passed
@piorpua piorpua deleted the officecli-readme-gifs branch March 30, 2026 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot:ready-to-merge Bot done, code is clean — human just needs to confirm and merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: update OfficeCLI assistant demos (PPT/Word/Excel)

2 participants