优化提现审核侧页基础信息展示 #91

2026-07-27T19:30:08+08:00

linkeyu commented

2026-07-27 19:30:08 +08:00

改动内容

删除提现详情侧页的提现类型
删除提现单区域重复展示的累计成功提现
用户详情和邀请详情中的累计成功提现继续保留
两类累计金额继续按 coin_cash / invite_cash 账户分别统计

验证

前端 npm run build 通过
后端账户隔离相关测试 2 passed
线上数据库抽查确认两类提现金额独立
git diff --check 通过

## 改动内容 - 删除提现详情侧页的提现类型 - 删除提现单区域重复展示的累计成功提现 - 用户详情和邀请详情中的累计成功提现继续保留 - 两类累计金额继续按 coin_cash / invite_cash 账户分别统计 ## 验证 - 前端 npm run build 通过 - 后端账户隔离相关测试 2 passed - 线上数据库抽查确认两类提现金额独立 - git diff --check 通过

linkeyu added 1 commit 2026-07-27 19:30:09 +08:00

优化：精简提现详情基础信息 c8c64846ad

guke commented

2026-07-28 10:00:42 +08:00

🤖 review-pr 深审结论 · 🟢 置信度 0.92

删掉详情侧栏两行 Descriptions.Item(提现类型、累计成功提现),纯删除(0 增 6 删),干净、类型安全。已在隔离 worktree 里与 #90 合并后跑 tsc --noEmit 通过(exit 0)。

正面点

累计成功提现 是重复展示:同一详情视图 L1160-1162 已有一张 Statistic 卡片展示同一个 detail.cumulative_success_cents,删这行是去重,合理。
两个被删字段别处仍在用 → 无死代码/无未用变量:cumulative_success_cents 还用于列表列(L545)、markNormalRisk(L1055)、统计卡(L1162);detail.order.source 还用于 L1196。
合并结果 tsc --noEmit 通过。

建议(low · 产品确认,非缺陷)

删除后「提现类型(邀请奖励/福利页)」在详情侧栏不再可见;但每页 source 由 mode 固定(L180 mode==='invite'?'invite_cash':'coin_cash'),同页订单类型本就一致、冗余度高。确认是有意精简即可。

由 review-pr 深审(有本地克隆 → worktree 完整仓上下文;已与 #90 试合并 + tsc 实跑验证)

## 🤖 review-pr 深审结论 · 🟢 置信度 0.92 删掉详情侧栏两行 `Descriptions.Item`(提现类型、累计成功提现),纯删除(0 增 6 删),干净、类型安全。已在隔离 worktree 里与 #90 合并后跑 `tsc --noEmit` 通过(exit 0)。 **正面点** - `累计成功提现` 是重复展示:同一详情视图 L1160-1162 已有一张 `Statistic` 卡片展示同一个 `detail.cumulative_success_cents`,删这行是去重,合理。 - 两个被删字段别处仍在用 → 无死代码/无未用变量:`cumulative_success_cents` 还用于列表列(L545)、`markNormalRisk`(L1055)、统计卡(L1162);`detail.order.source` 还用于 L1196。 - 合并结果 `tsc --noEmit` 通过。 **建议(low · 产品确认,非缺陷)** - 删除后「提现类型(邀请奖励/福利页)」在详情侧栏不再可见;但每页 `source` 由 `mode` 固定(L180 `mode==='invite'?'invite_cash':'coin_cash'`),同页订单类型本就一致、冗余度高。确认是有意精简即可。 --- _由 review-pr 深审(有本地克隆 → worktree 完整仓上下文;已与 #90 试合并 + tsc 实跑验证)_

guke referenced this pull request

2026-07-28 10:00:43 +08:00

优化提现审核页原因列展示 #90

guke merged commit 5acfb05c57 into main

2026-07-28 10:04:05 +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#91