fix(web): #558 修复bind-contact接口字段名，对齐后端BindContactRequest #708

Merged

auto-approve-bot merged 4 commits from fix/558-bind-contact-api-fields into develop 2026-07-22 14:12:57 +08:00

Conversation 5 Commits 4 Files Changed 3

+16 -17

xiaoxia commented 2026-07-22 13:57:36 +08:00

Owner

Copy Link

Copy Source

问题

后端 BindContactRequest 字段为 email / email_code / phone / phone_code，
前端之前传 target / value / code 导致 400 错误（"至少填写手机号或邮箱"）。

改动

• api/auth.ts：BindContactRequest 字段对齐后端
• BindContactModal.tsx：提交时按 email/email_code 或 phone/phone_code 组装参数

## 问题 后端 `BindContactRequest` 字段为 `email` / `email_code` / `phone` / `phone_code`， 前端之前传 `target` / `value` / `code` 导致 400 错误（"至少填写手机号或邮箱"）。 ## 改动 - `api/auth.ts`：BindContactRequest 字段对齐后端 - `BindContactModal.tsx`：提交时按 email/email_code 或 phone/phone_code 组装参数

xiaoxia added 4 commits 2026-07-22 13:57:36 +08:00

feat(web): #558 微信登录体验优化 - state校验/登录跳转/useWechatCallback hook ... fb73a69ecd

- 回调页增加state防CSRF校验
- 登录成功后跳回登录前页面（login_redirect）
- 新增useWechatCallback hook统一微信登录逻辑
- 绑定成功后更新authStore用户信息
- 移除重复的token存储逻辑（setAuth已处理）

test(web): 修复useLogin测试 - navigate带replace参数 + 新增login_redirect跳转测试 ... 2f2b2f77b6

- 登录成功navigate断言增加{ replace: true }
- 移除localStorage token断言（setAuth mock不真实存储）
- 新增login_redirect跳转测试用例

Merge branch 'develop' of https://git.xiaoxiajianji.com/xiaoxia/xiaoxia-saas into feat/558-wechat-login-polish 58db7ab6ec

fix(web): #558 修复bind-contact接口字段名，对齐后端BindContactRequest ... b082013bba

后端字段为email/email_code/phone/phone_code，
前端之前传target/value/code导致400错误。

gitea-actions bot commented 2026-07-22 13:59:29 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

项目	详情
PR号	#708
预览链接	https://pr-708.preview.xiaoxiajianji.com
API环境	staging

💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后，预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #708 | | 预览链接 | [https://pr-708.preview.xiaoxiajianji.com](https://pr-708.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。

auto-approve-bot approved these changes 2026-07-22 13:59:43 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-07-22 13:59:43 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot commented 2026-07-22 14:02:19 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #708

⚠️ 问题（2个需要修改）

1. apps/web/src/components/auth/BindContactModal.tsx 第71-73行：未检查业务逻辑成功标志直接提示成功

   • BindContactResponse 接口已更新为包含 success: boolean 字段，但代码在收到响应后直接执行 message.success 和 onSuccess，未校验 result.success。如果后端返回 HTTP 200 但业务处理失败（success: false），前端会错误地提示“绑定成功”并执行回调，误导用户。

2. apps/web/src/components/auth/BindContactModal.tsx 第64-73行：缺少异常处理导致 Loading 状态无法复位

   • 在调用 bindContact 前设置了 setLoading(true)，但未使用 try...catch 或 .finally() 包裹异步请求。如果 bindContact 抛出异常（如网络错误、500错误），函数执行中断，setLoading(false) 永远不会被执行，导致 Modal 按钮一直处于加载禁用状态。

💡 建议（1个可选）

1. apps/web/src/api/auth.ts 第151-155行：建议使用判别联合类型增强类型安全
   • 当前 BindContactRequest 接口所有字段均为可选，TypeScript 无法在编译期防止发送空对象 {} 或同时发送 email 和 phone 的非法请求。建议改用联合类型（如 type BindContactRequest = ({ type: 'email', email: string, code: string }) | ({ type: 'phone', phone: string, code: string })）以确保数据结构的互斥性和完整性。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-22 14:02:19 | 模型:}

## 代码审查结果 - PR #708 ### ⚠️ 问题（2个需要修改） 1. **apps/web/src/components/auth/BindContactModal.tsx 第71-73行**：未检查业务逻辑成功标志直接提示成功 - `BindContactResponse` 接口已更新为包含 `success: boolean` 字段，但代码在收到响应后直接执行 `message.success` 和 `onSuccess`，未校验 `result.success`。如果后端返回 HTTP 200 但业务处理失败（`success: false`），前端会错误地提示“绑定成功”并执行回调，误导用户。 2. **apps/web/src/components/auth/BindContactModal.tsx 第64-73行**：缺少异常处理导致 Loading 状态无法复位 - 在调用 `bindContact` 前设置了 `setLoading(true)`，但未使用 `try...catch` 或 `.finally()` 包裹异步请求。如果 `bindContact` 抛出异常（如网络错误、500错误），函数执行中断，`setLoading(false)` 永远不会被执行，导致 Modal 按钮一直处于加载禁用状态。 ### 💡 建议（1个可选） 1. **apps/web/src/api/auth.ts 第151-155行**：建议使用判别联合类型增强类型安全 - 当前 `BindContactRequest` 接口所有字段均为可选，TypeScript 无法在编译期防止发送空对象 `{}` 或同时发送 email 和 phone 的非法请求。建议改用联合类型（如 `type BindContactRequest = ({ type: 'email', email: string, code: string }) | ({ type: 'phone', phone: string, code: string })`）以确保数据结构的互斥性和完整性。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-22 14:02:19 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot merged commit b2700b85ff into develop 2026-07-22 14:12:57 +08:00

auto-approve-bot deleted branch fix/558-bind-contact-api-fields 2026-07-22 14:12:57 +08:00

gitea-actions bot commented 2026-07-22 14:13:07 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #708 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #708 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

Sign in to join this conversation.

Reviewers

No Reviewers

auto-approve-bot

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#708