在比价记录中展示是否下单 #88

2026-07-27T14:57:20+08:00

linkeyu commented

2026-07-27 14:57:20 +08:00

改动说明

在“省”列后新增“是否下单”列
使用“已下单/未下单”标签展示
同步补充前端接口类型

验证

Next.js 生产构建和类型检查通过

## 改动说明 - 在“省”列后新增“是否下单”列 - 使用“已下单/未下单”标签展示 - 同步补充前端接口类型 ## 验证 - Next.js 生产构建和类型检查通过

linkeyu added 1 commit 2026-07-27 14:57:21 +08:00

功能：展示比价记录是否下单 9df6a2d792

在比价记录的省金额列后新增是否下单列，并同步接口类型。

guke commented

2026-07-27 17:24:10 +08:00

🤖 review-pr 深审结论

🟢 可以合并（置信度 0.9）。改动极小且正确，类型检查通过（含与 main 的合并结果）。

已实跑验证（worktree 隔离副本，未污染工作区）：

tsc --noEmit → 通过（exit 0）。
⚠️ 关键：main 期间改过 src/lib/types.ts（55 行），与本 PR 同文件。已做试合并 origin/main 后再跑一次 tsc --noEmit → 仍通过，ordered: boolean 与 main 的改动无冲突、无重复定义。（只看分支快照会漏掉这层——合并后才编译不过的类型问题正是这步在挡。）
Tag 已在文件顶部从 antd 导入（既有多处使用），新列 render 无缺失导入。

写得对的地方：

dataIndex: 'ordered' 与后端 schema 字段名 ordered: bool 对齐；类型 ordered: boolean 与后端契约一致。
绿/默认 Tag 的「已下单 / 未下单」与既有状态列风格统一。

一条 FYI（非阻塞，low）：

前后端发布顺序：ordered 在类型里是必填 boolean。若前端先于后端（app-server #184）上线，接口暂不返回该字段 → 运行时为 undefined → 渲染成「未下单」。不会崩，但后端上线前该列会全显「未下单」。建议后端先合/先发，或与 #184 同批发布。

建议：与后端 #184 同批合并 / 发布即可。

## 🤖 review-pr 深审结论 🟢 **可以合并**（置信度 0.9）。改动极小且正确，类型检查通过（含与 main 的合并结果）。 **已实跑验证（worktree 隔离副本，未污染工作区）：** - `tsc --noEmit` → **通过**（exit 0）。 - ⚠️ 关键：`main` 期间改过 `src/lib/types.ts`（55 行），与本 PR 同文件。已做**试合并** `origin/main` 后**再跑一次 `tsc --noEmit`** → 仍**通过**，`ordered: boolean` 与 main 的改动无冲突、无重复定义。（只看分支快照会漏掉这层——合并后才编译不过的类型问题正是这步在挡。） - `Tag` 已在文件顶部从 `antd` 导入（既有多处使用），新列 render 无缺失导入。 **写得对的地方：** - `dataIndex: 'ordered'` 与后端 schema 字段名 `ordered: bool` 对齐；类型 `ordered: boolean` 与后端契约一致。 - 绿/默认 Tag 的「已下单 / 未下单」与既有状态列风格统一。 **一条 FYI（非阻塞，low）：** - **前后端发布顺序**：`ordered` 在类型里是必填 `boolean`。若前端先于后端（app-server #184）上线，接口暂不返回该字段 → 运行时为 `undefined` → 渲染成「未下单」。不会崩，但后端上线前该列会全显「未下单」。建议**后端先合/先发**，或与 #184 同批发布。 **建议**：与后端 #184 同批合并 / 发布即可。

guke merged commit bea35fb707 into main

2026-07-27 17:33:24 +08:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: WonderableAI/shaguabijia-admin-web#88