Skip to content

refactor(wallet): reuse core top-up lifecycle - #20

Merged
boompieman merged 1 commit into
releasefrom
agent/manual-bank-transfer-static
Aug 19, 2026
Merged

refactor(wallet): reuse core top-up lifecycle#20
boompieman merged 1 commit into
releasefrom
agent/manual-bank-transfer-static

Conversation

@boompieman

Copy link
Copy Markdown
Owner

⚠️ 提交说明 / PR Notice

Important

  • 请提供人工撰写的简洁摘要,避免直接粘贴未经整理的 AI 输出。
  • 本次代码与 PR 描述由 Codex AI 协助产生,并由仓库维护者发起合并。

📝 变更描述 / Description

将手动汇款收敛为官方充值订单生命周期上的薄层适配:

  • 移除手动汇款在 model 层新增的付款常数,改为功能文件内局部常数。
  • 不再为 /topup/info 增加专用启用字段;钱包直接从既有 pay_methods 判断是否可用。
  • 建立订单入口归入 /api/user/topup/manual-bank-transfer,只负责建立标准 pending TopUp
  • 管理员查询、完成订单、增加额度及幂等保护继续完全沿用官方既有 AdminCompleteTopUp / ManualCompleteTopUp 流程。

这样减少对官方 Core 的修改面,并避免重复实现已由 QuantumNous#1976 提供的订单完成机制。

🚀 变更类型 / Type of change

  • 🐛 Bug 修复 (Bug fix) - 请关联对应 Issue,避免将设计取舍、理解偏差或预期不一致直接归类为 bug
  • ✨ 新功能 (New feature) - 重大特性建议先通过 Issue 沟通
  • ⚡ 性能优化 / 重构 (Refactor)
  • 📝 文档更新 (Documentation)

🔗 关联任务 / Related Issue

✅ 提交前检查项 / Checklist

  • 人工确认: 我已亲自整理并撰写此描述,没有直接粘贴未经处理的 AI 输出。(本次明确标示为 AI-assisted)
  • 非重复提交: 我已搜索现有的 IssuesPRs,确认不是重复提交。
  • Bug fix 说明: 此 PR 不标记为 Bug fix。
  • 变更理解: 已确认手动汇款只建立 pending 订单,管理员完成后才增加额度。
  • 范围聚焦: 本 PR 未包含任何与当前任务无关的代码改动。
  • 本地验证: 已运行并通过相关测试与建置。
  • 安全合规: 代码中无敏感凭据,且符合项目代码规范。

📸 运行证明 / Proof of Work

  • go test ./controller -count=1
  • Wallet Vitest:4 files / 11 tests passed
  • TypeScript typecheck passed
  • Modified-file oxlint passed
  • Rsbuild production build passed

@boompieman
boompieman merged commit eee7775 into release Aug 19, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant