修正退款为独立一级目录 #31

Merged
renshen merged 2 commits from fix/renshen-refund-top-level-navigation into main 2026-08-11 16:28:42 +08:00
Collaborator

任务占用与边界

  • 任务名称:修正退款为独立一级目录
  • 实际开发者:renshen
  • 任务来源/团队分工:用户截图纠正,退款应与订单数据并列
  • 目标结果:从“订单数据”的二级列表移除“退款”,并在“订单数据”和“店铺应收账款”之间增加独立一级目录“退款”
  • 预计修改路径:config/business_navigation.php;tests/Feature/Settings/NavigationTest.php;tests/Feature/Business/PagesTest.php
  • 将写入的公共契约:删除 order-data/refunds 子菜单契约;新增 refunds 一级菜单键
  • 仅复用、明确不修改的公共契约:business.pages.show;Business\Pages::show() 通用页面分发;标准空状态
  • 数据源、表/视图和字段:无
  • 上游 source_key / scope:无
  • 权限名、菜单键或共享组件:refunds 一级菜单;不修改权限和共享组件
  • 明确不修改的范围:app/Http/Controllers/Business/Pages.php、routes/、Repository、上游查询、权限、迁移、接口及两个订单数据报表

开工碰撞检查

  • 已执行 scripts/codex-task-preflight.ps1
  • 已检查最新 main 和所有远程任务分支
  • 已搜索主线既有页面、接口、Repository 和数据查询
  • 已区分写入与仅复用契约
  • 未发现其他远程任务分支写入重叠

相关分支、重叠项与协调结论:远端仅有 origin/main;本任务直接纠正 main 中上一版 order-data/refunds 层级,不创建重复入口。

实际改动

  • 完成内容:待完成
  • Codex 实际修改的文件:待完成
  • 新增或变更的公共契约:待完成
  • 与原计划的差异:待完成

验证证据

  • 正常流程:待完成
  • 空数据/错误输入/无权限场景:待完成
  • 受影响旧功能:待完成
  • 自动测试及结果:待完成
  • 实际页面结果:待完成

风险检查

  • 不涉及金额、权限、审批、数据库迁移、生产配置或数据修复
  • 未提交敏感信息、真实财务数据、日志或构建产物
  • 前端改动复用现有一级导航与标准空状态,符合 DESIGN.md

高风险影响、回滚和授权状态:不涉及高风险操作,不部署生产。

合并前复查

  • 已重新拉取远程引用并用实际范围再次运行碰撞检查
  • PR 不再是 WIP,所有测试结果已补齐
  • 已逐文件审核 origin/main...任务分支完整 diff
  • 自审确认无越界修改、重复实现、敏感信息、冲突或未验证结果
  • 合并顺序不会覆盖其他未合并任务
  • 已授权当前任务 Codex 执行 Squash merge、验证 main 并清理任务分支
## 任务占用与边界 - 任务名称:修正退款为独立一级目录 - 实际开发者:renshen - 任务来源/团队分工:用户截图纠正,退款应与订单数据并列 - 目标结果:从“订单数据”的二级列表移除“退款”,并在“订单数据”和“店铺应收账款”之间增加独立一级目录“退款” - 预计修改路径:config/business_navigation.php;tests/Feature/Settings/NavigationTest.php;tests/Feature/Business/PagesTest.php - 将写入的公共契约:删除 order-data/refunds 子菜单契约;新增 refunds 一级菜单键 - 仅复用、明确不修改的公共契约:business.pages.show;Business\Pages::show() 通用页面分发;标准空状态 - 数据源、表/视图和字段:无 - 上游 source_key / scope:无 - 权限名、菜单键或共享组件:refunds 一级菜单;不修改权限和共享组件 - 明确不修改的范围:app/Http/Controllers/Business/Pages.php、routes/、Repository、上游查询、权限、迁移、接口及两个订单数据报表 ## 开工碰撞检查 - [x] 已执行 scripts/codex-task-preflight.ps1 - [x] 已检查最新 main 和所有远程任务分支 - [x] 已搜索主线既有页面、接口、Repository 和数据查询 - [x] 已区分写入与仅复用契约 - [x] 未发现其他远程任务分支写入重叠 相关分支、重叠项与协调结论:远端仅有 origin/main;本任务直接纠正 main 中上一版 order-data/refunds 层级,不创建重复入口。 ## 实际改动 - 完成内容:待完成 - Codex 实际修改的文件:待完成 - 新增或变更的公共契约:待完成 - 与原计划的差异:待完成 ## 验证证据 - 正常流程:待完成 - 空数据/错误输入/无权限场景:待完成 - 受影响旧功能:待完成 - 自动测试及结果:待完成 - 实际页面结果:待完成 ## 风险检查 - [x] 不涉及金额、权限、审批、数据库迁移、生产配置或数据修复 - [x] 未提交敏感信息、真实财务数据、日志或构建产物 - [x] 前端改动复用现有一级导航与标准空状态,符合 DESIGN.md 高风险影响、回滚和授权状态:不涉及高风险操作,不部署生产。 ## 合并前复查 - [ ] 已重新拉取远程引用并用实际范围再次运行碰撞检查 - [ ] PR 不再是 WIP,所有测试结果已补齐 - [ ] 已逐文件审核 origin/main...任务分支完整 diff - [ ] 自审确认无越界修改、重复实现、敏感信息、冲突或未验证结果 - [ ] 合并顺序不会覆盖其他未合并任务 - [x] 已授权当前任务 Codex 执行 Squash merge、验证 main 并清理任务分支
renshen added 1 commit 2026-08-11 15:59:28 +08:00
renshen added 1 commit 2026-08-11 16:21:43 +08:00
Author
Collaborator

合并前自审完成:

  • 原始要求:退款与订单数据并列,并位于订单数据和店铺应收账款之间。
  • 完整差异仅包含 config/business_navigation.php、tests/Feature/Settings/NavigationTest.php、tests/Feature/Business/PagesTest.php。
  • 未修改 Controller、路由、Repository、权限、数据库、迁移或上游查询。
  • 复用现有 business.pages.show 通用页面逻辑;新地址 /1/workspace/refunds 正常返回标准空状态,旧嵌套地址返回 404。
  • 容器内以 www-data 执行相关测试:11 passed,171 assertions。
  • git diff --check 通过;未发现敏感信息、无关文件或远端分支重叠。
  • 实际页面验证:退款显示为无展开箭头的独立一级菜单,顺序正确。

风险:仅改变退款入口 URL;当前尚无退款业务数据功能。本次不涉及生产部署、数据库迁移或数据写入。

合并前自审完成: - 原始要求:退款与订单数据并列,并位于订单数据和店铺应收账款之间。 - 完整差异仅包含 `config/business_navigation.php`、`tests/Feature/Settings/NavigationTest.php`、`tests/Feature/Business/PagesTest.php`。 - 未修改 Controller、路由、Repository、权限、数据库、迁移或上游查询。 - 复用现有 `business.pages.show` 通用页面逻辑;新地址 `/1/workspace/refunds` 正常返回标准空状态,旧嵌套地址返回 404。 - 容器内以 `www-data` 执行相关测试:11 passed,171 assertions。 - `git diff --check` 通过;未发现敏感信息、无关文件或远端分支重叠。 - 实际页面验证:退款显示为无展开箭头的独立一级菜单,顺序正确。 风险:仅改变退款入口 URL;当前尚无退款业务数据功能。本次不涉及生产部署、数据库迁移或数据写入。
renshen changed title from WIP: 修正退款为独立一级目录 to 修正退款为独立一级目录 2026-08-11 16:28:17 +08:00
renshen merged commit b3594e7563 into main 2026-08-11 16:28:42 +08:00
renshen deleted branch fix/renshen-refund-top-level-navigation 2026-08-11 16:28:43 +08:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: WuYongJie/CaiWuXiTong#31
No description provided.