优化比价记录设备信息展示 #87

Merged
guke merged 2 commits from codex/comparison-device-display into main 2026-07-27 16:38:35 +08:00
Member

改动说明

  • 机型优先显示可读商品名,同时保留原始设备编码
  • 列表和详情展示 ROM 厂商、名称与版本
  • 同步补充前端接口类型

验证

  • Next.js 生产构建通过
  • 本地页面列表与详情抽屉已检查
## 改动说明 - 机型优先显示可读商品名,同时保留原始设备编码 - 列表和详情展示 ROM 厂商、名称与版本 - 同步补充前端接口类型 ## 验证 - Next.js 生产构建通过 - 本地页面列表与详情抽屉已检查
linkeyu added 1 commit 2026-07-27 14:37:35 +08:00
列表和详情优先展示可读机型名,保留原始设备编码,并补充 ROM 版本。
linkeyu changed title from ???????????? to 优化比价记录设备信息展示 2026-07-27 14:38:16 +08:00
Member

🤖 review-pr 深审结论

🟢 可以合(置信度 0.9)。已在隔离 worktree 实跑验证:tsc --noEmit(strict)exit 0;对当前 main(63538dd)干净合入;与后端 app-server #183 契约完全对称。

改动

  • 抽出 deviceName / romName / renderDevice;列表「机型/ROM」列改为 可读名 + 原始编码(仅当与可读名不同才显示,避免重复) + ROM 串,列宽 150→180;详情「设备环境」卡片同步。
  • types.ts:ComparisonRecordListItem 加 device_model_name + rom_version,ComparisonRecordDetail 去掉 rom_version(继承),与后端一致。

落地/部署顺序

  • 依赖后端 #183 提供 device_model_name / rom_version(当前 main 尚无),建议后端 #183 先上。即便前端先上也是优雅降级:缺 device_model_name → 回退原始机型,缺 rom_version → ROM 串省略,不会崩。
  • 兄弟 PR #88(展示是否下单)改同 2 文件,但实测自动合并无冲突(不同列/不同字段),无需担心。

搭配后端 app-server #183。

## 🤖 review-pr 深审结论 🟢 **可以合**(置信度 0.9)。已在隔离 worktree 实跑验证:`tsc --noEmit`(strict)exit 0;对当前 main(63538dd)干净合入;与后端 app-server #183 契约完全对称。 **改动** - 抽出 deviceName / romName / renderDevice;列表「机型/ROM」列改为 可读名 + 原始编码(仅当与可读名不同才显示,避免重复) + ROM 串,列宽 150→180;详情「设备环境」卡片同步。 - types.ts:ComparisonRecordListItem 加 device_model_name + rom_version,ComparisonRecordDetail 去掉 rom_version(继承),与后端一致。 **落地/部署顺序** - 依赖后端 #183 提供 device_model_name / rom_version(当前 main 尚无),建议后端 #183 先上。即便前端先上也是优雅降级:缺 device_model_name → 回退原始机型,缺 rom_version → ROM 串省略,不会崩。 - 兄弟 PR #88(展示是否下单)改同 2 文件,但实测自动合并无冲突(不同列/不同字段),无需担心。 搭配后端 app-server #183。
guke added 1 commit 2026-07-27 16:38:33 +08:00
guke merged commit 93c3daa784 into main 2026-07-27 16:38:35 +08:00
Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: WonderableAI/shaguabijia-admin-web#87