cykj/zkxw
1
0 Fork 0
Code Issues 7 Pull Requests 1 Packages Projects Releases Wiki Activity

fix(oa): 审批单 PDF 补齐评论、提及及附件链接 #35

Merged
cykj merged 1 commits from codex/issue-30-oa-print-comments into V2.0 2026-11-05 02:40:22 +00:00
Conversation 0 Commits 1 Files Changed 7
+355 -28
cykj commented 2026-11-05 02:29:09 +00:00
Owner
Copy link
Copy source

当前状态：按用户2026-11-05最新指令先合并到 V2.0，再由用户测试。已合并，验收仍待用户确认，工单 #30 保持开放。

Summary

OA 审批单 PDF 原先只打印审批节点，遗漏讨论评论。Web 与小程序承载的移动 OA 现在沿用详情页时间顺序，打印评论作者、时间、正文、@人员及附件链接；撤回评论只显示撤回状态。

 审批流程打印
   审批节点与审批意见
+ 讨论评论、提及人员、附件 URL
+ 附件对应的 PDF 可点击注解

关联工单：#30

用户已确认范围：web-admin、mobile-h5、mobile-oa-h5。小程序入口代码无需更改，实际修复位于 web-admin 和 mobile-oa-h5；后端未改动。复用原打印表格 OaApprovalPrintSheet.vue，属于缺失数据修复，无新增界面或视觉改版，P01–P27 页面模式不适用。

Evidence

  • Before: 两端新增评论回归用例均失败：期望4条流程记录，实际2条；原导出 PDF 不含附件链接注解。
    After: Web 5项测试、移动 OA 7项测试通过，覆盖同秒排序、纯附件/提及、撤回保护及既有流程。
  • 合成数据离线浏览器实际导出：Web/移动 PDF 均显示评论与转义文本，撤回内容不可见；短链接各1个可点击注解。长评论生成3页 PDF，核对换行及跨页链接注解，MinIO相对地址解析为绝对URL。
  • 移动 OA 最终构建通过；用户改为自行测试后，Web 全量构建已停止，未宣称通过。
  • 独立审查发现并修正同秒排序问题，复审无新增阻塞项。
  • 当前提交：a511004812b2f7ca13a6e54557b253a57d06bf6e。
  • 待用户验收： 用户在当前会话明确“这个我来测试”，Agent 已停止后续测试与构建；须用户确认任务版本及集成环境通过后才合并关单。 真实前后端集成环境及微信 WebView 打印链路待验证。共享环境由工单 #13 占用，Agent 未启停服务或切换他人环境；必要验证前不合并、不关闭工单。

Merge Danger

Door: two-way

纯前端打印修复，可回退，无数据库/API变更。

Blast Radius: OA打印

新增评论会增加 PDF 页数；沿用已有分页方式和附件访问权限。

当前状态：按用户2026-11-05最新指令先合并到 V2.0，再由用户测试。已合并，验收仍待用户确认，工单 #30 保持开放。 ## Summary OA 审批单 PDF 原先只打印审批节点，遗漏讨论评论。Web 与小程序承载的移动 OA 现在沿用详情页时间顺序，打印评论作者、时间、正文、@人员及附件链接；撤回评论只显示撤回状态。 ```diff 审批流程打印 审批节点与审批意见 + 讨论评论、提及人员、附件 URL + 附件对应的 PDF 可点击注解 ``` 关联工单：https://gitea.aichengyan.com/cykj/zkxw/issues/30 用户已确认范围：web-admin、mobile-h5、mobile-oa-h5。小程序入口代码无需更改，实际修复位于 web-admin 和 mobile-oa-h5；后端未改动。复用原打印表格 OaApprovalPrintSheet.vue，属于缺失数据修复，无新增界面或视觉改版，P01–P27 页面模式不适用。 ## Evidence - **Before:** 两端新增评论回归用例均失败：期望4条流程记录，实际2条；原导出 PDF 不含附件链接注解。 **After:** Web 5项测试、移动 OA 7项测试通过，覆盖同秒排序、纯附件/提及、撤回保护及既有流程。 - 合成数据离线浏览器实际导出：Web/移动 PDF 均显示评论与转义文本，撤回内容不可见；短链接各1个可点击注解。长评论生成3页 PDF，核对换行及跨页链接注解，MinIO相对地址解析为绝对URL。 - 移动 OA 最终构建通过；用户改为自行测试后，Web 全量构建已停止，未宣称通过。 - 独立审查发现并修正同秒排序问题，复审无新增阻塞项。 - 当前提交：a511004812b2f7ca13a6e54557b253a57d06bf6e。 - **待用户验收：** 用户在当前会话明确“这个我来测试”，Agent 已停止后续测试与构建；须用户确认任务版本及集成环境通过后才合并关单。 真实前后端集成环境及微信 WebView 打印链路待验证。共享环境由工单 #13 占用，Agent 未启停服务或切换他人环境；必要验证前不合并、不关闭工单。 ## Merge Danger **Door:** two-way 纯前端打印修复，可回退，无数据库/API变更。 **Blast Radius:** OA打印 新增评论会增加 PDF 页数；沿用已有分页方式和附件访问权限。
cykj added 1 commit 2026-11-05 02:29:09 +00:00
fix(oa): include discussion comments and attachment links in approval PDFs a511004812 
Merge discussion comments into the existing printed approval flow on Web and mobile OA, preserving detail timeline order and hiding revoked content. Include mentions and absolute attachment URLs, and add PDF link annotations so rasterized exports retain usable links. Add regression tests for ordering, attachments, mentions and revoked comments. Related to Gitea issue #30.
cykj merged commit 6292badf8e into V2.0 2026-11-05 02:40:22 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
ai-done
ai-in-progress
ai-merging
ai-ready
ai-ready-to-merge
ai-testing
ai-waiting-user
已修复
修复代码和验证结果已提交；验收、合并和发布状态见工单。
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
cykj
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: cykj/zkxw#35