优化用户反馈设备信息展示 #85

2026-07-27T10:56:01+08:00

linkeyu commented

2026-07-27 10:56:01 +08:00

改动说明

反馈列表优先展示可读手机型号
同时展示原始硬件编码与厂商信息
厂商系统版本和 Android 版本分层展示

验证

Next.js 生产构建通过

## 改动说明 - 反馈列表优先展示可读手机型号 - 同时展示原始硬件编码与厂商信息 - 厂商系统版本和 Android 版本分层展示 ## 验证 - Next.js 生产构建通过

linkeyu added 1 commit 2026-07-27 10:56:02 +08:00

优化：展示反馈机型与系统版本 8d75a6fde3

反馈列表展示可读手机型号、原始硬件编码、厂商系统版本和 Android 版本。

guke commented

2026-07-27 12:04:39 +08:00

🤖 review-pr 深审结论（前端 · admin-web #85）

配套后端 app-server #181。本仓无本地克隆，走 API 降级：已拉 head 全量 page.tsx + types.ts 逐场景走查。

总评 🟢 可合：类型新增与后端 FeedbackOut 完全对齐（device_model_name/device_manufacturer: string|null、rom_version: number|null）。置信度 ~0.85。

走查结论

renderDeviceOs 四种场景均渲染正确：已知机型+命中比价记录 / 已知+未命中 / 未知编码 / 历史 null。
romHasVersion 检测 rom_name 是否已含数字，避免「MIUI 14 14」重复拼版本，考虑周到。
device_model 归一化、列宽 150→180 配合两行副信息合理。

一个跨仓提示（主改动在后端）

后端对历史反馈（device_model=None）会省略 device_model_name/device_manufacturer/rom_version 三个 key（不是 null，是键不存在）。本前端因 || / && / .filter(Boolean) 防御式写法，undefined 与 null 表现一致，当前不受影响；仅提示这三个字段并非总是存在，勿在别处假设它一定是 null。详见后端 #181 评论。

—— 自动深审，仅供参考，具体以人工判断为准。

## 🤖 review-pr 深审结论（前端 · admin-web #85）配套后端 app-server #181。本仓无本地克隆，走 API 降级：已拉 head 全量 `page.tsx` + `types.ts` 逐场景走查。 **总评 🟢 可合**：类型新增与后端 `FeedbackOut` 完全对齐（`device_model_name`/`device_manufacturer: string|null`、`rom_version: number|null`）。置信度 ~0.85。 ### 走查结论 - `renderDeviceOs` 四种场景均渲染正确：已知机型+命中比价记录 / 已知+未命中 / 未知编码 / 历史 null。 - `romHasVersion` 检测 `rom_name` 是否已含数字，避免「MIUI 14 14」重复拼版本，考虑周到。 - `device_model` 归一化、列宽 150→180 配合两行副信息合理。 ### 一个跨仓提示（主改动在后端） - 后端对历史反馈（`device_model=None`）会**省略** `device_model_name`/`device_manufacturer`/`rom_version` 三个 key（不是 `null`，是键不存在）。本前端因 `||` / `&&` / `.filter(Boolean)` 防御式写法，`undefined` 与 `null` 表现一致，**当前不受影响**；仅提示这三个字段并非总是存在，勿在别处假设它一定是 `null`。详见后端 #181 评论。 —— 自动深审，仅供参考，具体以人工判断为准。

guke merged commit 10b82ad564 into main

2026-07-27 13:33:55 +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#85