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

过滤严谨:只有该列显式设了 key,其余列 key===undefined,filter(c => c.key !== 'fail_reason') 恰好只摘这一列,其它列全保留。
精准命中目标表:columns 仅被主表使用(L953),inviteeColumns 是另一张独立表,未误改。
类型安全:column.key 是 antd ColumnType 合法字段,.filter 返回同型 ColumnsType。

待确认(med · 产品/逻辑) L651-654

只对 pending/success 隐藏,但状态页还有 reviewing(待审核)、rejected、failed、all。reviewing 是默认落地 tab(activeStatus 初值即 'reviewing',L159),待审核订单同样没有失败/拒绝原因,却仍会展示一整列全是「-」。本 PR 既是优化原因列展示,是否应把 reviewing 一并隐藏?请确认是有意保留还是漏了。

nit(可忽略)

visibleColumns 每次渲染重算、未 useMemo;但 columns 本身已每渲染重建,开销可忽略,不必改。

交叉:#90 与 #91 改动区域相隔很远(#90 在 ~570/650/950,#91 在 ~990),合并无冲突、合并后 tsc 通过,可任意顺序合入。

由 review-pr 深审(有本地克隆 → worktree 完整仓上下文 + tsc 实跑验证)

## 🤖 review-pr 深审结论 · 🟢 置信度 0.9(含 1 个待确认项) 给「失败/拒绝原因」列加 `key: 'fail_reason'`,并在 `activeStatus` 为 `pending`/`success` 时过滤掉该列。逻辑正确、类型安全、`tsc` 通过。 **正面点** - 过滤严谨:只有该列显式设了 `key`,其余列 `key===undefined`,`filter(c => c.key !== 'fail_reason')` 恰好只摘这一列,其它列全保留。 - 精准命中目标表:`columns` 仅被主表使用(L953),`inviteeColumns` 是另一张独立表,未误改。 - 类型安全:`column.key` 是 antd `ColumnType` 合法字段,`.filter` 返回同型 `ColumnsType`。 **待确认(med · 产品/逻辑) L651-654** - 只对 `pending`/`success` 隐藏,但状态页还有 `reviewing`(待审核)、`rejected`、`failed`、`all`。**`reviewing` 是默认落地 tab**(`activeStatus` 初值即 `'reviewing'`,L159),待审核订单同样没有失败/拒绝原因,却仍会展示一整列全是「-」。本 PR 既是优化原因列展示,是否应把 `reviewing` 一并隐藏?请确认是有意保留还是漏了。 **nit(可忽略)** - `visibleColumns` 每次渲染重算、未 `useMemo`;但 `columns` 本身已每渲染重建,开销可忽略,不必改。 --- 交叉:#90 与 #91 改动区域相隔很远(#90 在 ~570/650/950,#91 在 ~990),**合并无冲突、合并后 `tsc` 通过**,可任意顺序合入。 _由 review-pr 深审(有本地克隆 → worktree 完整仓上下文 + tsc 实跑验证)_

guke added 1 commit 2026-07-28 10:04:18 +08:00

Merge branch 'main' into codex/hide-withdraw-reason-column 4bf24cb6fd

guke merged commit 0040c5795d into main

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