From 54cdd4dca879726aeeb8423e503059be8e3206a5 Mon Sep 17 00:00:00 2001 From: API Changelog Bot Date: Sun, 13 Sep 2026 14:05:10 +0800 Subject: [PATCH] docs(changelog): document supplier revoke failure (#7487) --- ...撤销在途审批失败显式提示-修改接口-管理后台.md | 197 ++++++++++++++++++ 1 file changed, 197 insertions(+) create mode 100644 changelogs-v2/2026-09/13_7487_供应商撤销在途审批失败显式提示-修改接口-管理后台.md diff --git a/changelogs-v2/2026-09/13_7487_供应商撤销在途审批失败显式提示-修改接口-管理后台.md b/changelogs-v2/2026-09/13_7487_供应商撤销在途审批失败显式提示-修改接口-管理后台.md new file mode 100644 index 00000000..fd26ee99 --- /dev/null +++ b/changelogs-v2/2026-09/13_7487_供应商撤销在途审批失败显式提示-修改接口-管理后台.md @@ -0,0 +1,197 @@ +--- +schema: "hl-changelog/v2" +ticket: "7487" +title: "供应商撤销在途审批失败显式提示" +consumer: "admin" +author: "wx(GIT)" +change_type: "修改接口" +backend_status: "deployed" +gateway_status: "verified" +frontend_status: "pending" +frontend_owner: "mmg" +frontend_ref: "" +target_release: "" +verified_at: "2026-09-13" +status_note: "PR #7630 已合并 dev-v3，TEST 部署 merge 83626f88f；真实 Gateway 验证 HTTP 200 + 业务码 395029，失败不改本地状态。管理端消费确认待 mmg 完成。" +updated_at: "2026-09-13" +base: "dev-v3" +--- + +# 供应商撤销在途审批失败显式提示 + +供应商管理端撤销在途企微注册审批时，不再把企微未生效误报为成功。接口结构、HTTP 状态和 Feign 签名不变；调用方必须读取 `Result.code`。 + +> **服务**: `hl-resource-service`、`hl-user-service` +> **PR**: [#7630](https://git.1814.love:8443/wx/HL/pulls/7630) +> **Issue**: [#7487](https://git.1814.love:8443/wx/HL/issues/7487) +> **Merge commit**: [`83626f88f`](https://git.1814.love:8443/wx/HL/commit/83626f88f18cd3bfce21f462d9aaaad5b375d32f) + +## ⚠️ 关键变化 + +- 企微未生效时，管理端撤销返回业务码 `395029`、文案「企业微信撤销下发失败」。 +- user-service 不再请求不存在的 `/oa/approval/revoke`，改为返回明确失败和申请人操作指引。 +- `AUTO-*`、空单号与企微未配置路径保持原行为。 +- 企业微信没有服务端撤销 API；申请人需在企微客户端撤销，既有回调/轮询收到 `spStatus=4` 后自动退回 `DRAFT`。 + +## 二、变更接口清单 + +| # | 接口 | 方法 | 路径 | 变更类型 | 说明 | +|---|------|------|------|----------|------| +| 1 | 管理端撤销供应商注册审批 | POST | `/admin/supplier/items/{supplierId}/approval/revoke` | 返回语义修改 | 撤销未生效时返回业务错误 `395029`，不写本地成功态 | + +统一响应仍为 `Result`。HTTP 仍为 200；前端与 Feign 消费方必须判断业务字段 `code` / `success`。 + +## 三、接口详情 + +### 1. 管理端撤销供应商注册审批 `POST /admin/supplier/items/{supplierId}/approval/revoke` + +**VO**: `无请求 VO → Result` + +#### 使用场景 + +管理端供应商详情在供应商处于 `VETTING`、且存在企微在途注册审批时调用，用于请求撤销该审批。企业微信不支持服务端撤销时，本接口明确返回失败，前端引导申请人在企微客户端操作。 + +#### 入参 + +| 字段 | 位置 | 类型 | 必填 | 约束 | 说明 | +|------|------|------|------|------|------| +| `supplierId` | Path | Long | 是 | 正整数；供应商必须存在 | 待撤销审批的供应商 ID | +| 请求体 | Body | Object | 否 | 无业务字段 | 可发送空对象 `{}` | + +#### 出参 `Result` + +| 字段 | 类型 | 说明 | +|------|------|------| +| `code` | Integer | `200` 表示下发成功；`395029` 表示企微撤销未生效 | +| `message` | String | 成功文案或「企业微信撤销下发失败」 | +| `data` | Void | 固定为 `null` | +| `success` | Boolean | 仅 `code=200` 时为 `true` | + +#### 请求示例 + +```json +{} +``` + +#### 响应示例 + +下发成功时： + +```json +{ + "code": 200, + "message": "成功", + "data": null, + "success": true +} +``` + +#### 空数据 / 降级响应 + +本接口没有列表或空对象降级语义。user-service 返回空 `Result`、业务非 200 或 Feign 传输异常时，统一升级为 `395029`；不得降级成成功。 + +```json +{ "code": 395029, "message": "企业微信撤销下发失败", "data": null, "success": false } +``` + +#### 错误响应 + +```json +{ + "code": 395029, + "message": "企业微信撤销下发失败", + "data": null, + "traceId": null, + "success": false +} +``` + +#### 业务边界 + +- 需要已登录管理端身份和供应商更新/提交权限；未登录由 Gateway 返回 401。 +- 仅 `VETTING` 且存在 `PROFILE_CREATE + WECOM + PENDING` 在途记录可调用；供应商不存在、状态不符或无在途记录时返回既有业务错误。 +- 本次调用不直接把供应商改为 `DRAFT`；真实撤销终态仅由企微 `spStatus=4` 回调/轮询统一收口。 +- 企微撤销未生效、空响应或传输异常均返回 `395029`，并保证 `supplier_main` 与 `supplier_approval_log` 零写入。 +- HTTP 仍为 200，调用方必须判断 `Result.code` / `success`；重复失败重试不会伪造本地成功态。 + +## 四、契约约束与正确调用方式 + +调用方必须同时判断 HTTP 状态与 `Result.code` / `success`；本接口的业务失败仍使用 HTTP 200。 + +### 错误码与前端处理 + +| code | message | 触发条件 | 管理端处理 | +|---|---|---|---| +| `395029` | `企业微信撤销下发失败` | resource 调用 user 得到非 200、空响应或传输异常 | 不得提示撤销成功；提示失败并引导申请人到企微客户端撤销 | + +管理端建议同时展示在途审批单号；客户端撤销后等待回调/轮询自动退回草稿。 + +### 契约与兼容性 + +- Controller mapping、HTTP method、path 参数、`Result`、Feign 方法签名均未变化。 +- 无 DTO / VO 字段增删，无共享 `hl-common` 契约修改。 +- 这是内部 Feign 的行为语义变更；producer 与 resource consumer 定向测试均已执行。 +- Swagger 2 未配置 `oasdiff`，本单没有 OpenAPI 结构变化。 +- 未配置 Spring Cloud Contract；以 producer/consumer 源码审查、定向测试、全量测试和 TEST 实请求作为回退证据。 +- `hl-product-service-v2` 现有调用仍忽略 `Result` 并可能记录「审批已撤销」；按 #7487 范围只记录风险、不修改，由 `wx` 决定是否另立工单。 + +## 五、数据库行为 + +- 撤销下发失败时禁止修改 `supplier_main` 和 `supplier_approval_log`。 +- TEST 隔离夹具实测：调用前后供应商保持 `VETTING`，审批保持 `PENDING/NOT_APPLIED`，时间戳未变。 +- 本单无 DDL、无生产数据操作；TEST 临时夹具按批准 manifest 精确清理为 0 残留。 + +## 六、边界行为 + +- 空 `spNo` 仍返回 400；`AUTO-*` 仍成功且不调用企微；企微未配置仍返回原 500。 +- 仅真实企微单撤销未生效路径由静默成功改为明确失败。 +- 用户在企微客户端撤销后的 `spStatus=4` 回调/轮询收口不变。 + +## 六.6、修改前后对比 + +| 行为 | 修改前 | 修改后 | +|---|---|---| +| user-service 实际企微撤销失败 | `Result.code=200` | `Result.code=500` + 申请人操作指引 | +| resource-service 收到非 200 / 空响应 / 传输异常 | 非 200 仅日志、异常误用 395021 | 统一抛业务码 `395029` | +| 本地状态 | 不应修改 | 仍不修改，并有 DB 前后证据 | + +## 六.7、影响评估 + +- 管理端 supplier revoke → resource-service → user-service 的失败结果不再丢失。 +- 撤销失败不会修改本地供应商或审批状态。 +- 新码不进入 `SUPPLIER_WECOM_SUBMIT_FAILED(395021)` 的恢复逻辑；其余提交路径和文案未改。 +- 前端需要按业务码处理 `395029`，不可只看 HTTP 200。 + +## 七、不影响范围 + +- 不修改 `hl-common/**`、`hl-product-service-v2`、order-v3、house、groupbatch 或 `GroupVehicle*`。 +- 不修改企微回调/轮询和 `spStatus=4` 的既有退回草稿收口。 +- 不修改空 `spNo`、`AUTO-*`、企微未配置或真实成功路径。 +- 不涉及生产部署或生产数据库。 + +## 八、测试环境已验证 + +- user producer/client：77 tests；resource consumer：6 tests；product-v2 回归：35 tests；全部 0 failures/errors。 +- `mvn -o -pl hl-user-service -am test`：3990 tests，0 failures/errors。 +- `mvn -o -pl hl-resource-service -am test`：2902 tests，0 failures/errors。 +- TEST：resource/user 均为 `dev-v3@83626f88f`、`BEHIND=0`、`STATE=ok`。 +- Gateway：HTTP 200 + `code=395029`；前后 SELECT 证明状态未变。 +- 隔离夹具按 `hl-data-cleanup/v1` manifest 创建并精确清理，最终 0 残留。 +- Gateway jar mtime 后 `git log origin/dev-v3 -- hl-gateway` 无提交，无需重部署。 + +### 前端验收建议 + +1. `VETTING` 且有在途企微单时点击撤销。 +2. 收到 HTTP 200、`code=395029` 时展示错误，不得 toast 撤销成功。 +3. 提示申请人到企微客户端撤销，并说明回调/轮询会自动退回草稿。 +4. `AUTO-*` 仍按成功处理。 + +## 十、相关文档 + +- Issue [#7487](https://git.1814.love:8443/wx/HL/issues/7487) +- PR [#7630](https://git.1814.love:8443/wx/HL/pulls/7630) + +## 关联 / 联系人 + +- 后端负责人：@wx +- 前端负责人：@mmg