fix(wle): 推送明细第二列改为快递单号
业务二次要求(2026-10-22):明细里显示快递单号,不要内部单号。 - buildDetailLine 第二段从 `oId` 改为 `trackingNo`(上游 `logistic_id`) - 未揽收 / 无物流单的包裹本来就没有快递单号 → 退回 `oId`, 否则这条在群里无法定位(不让它变成「无单号」) - 单测:样例带 trackingNo 并断言内部单号不出现;补「无 trackingNo 时退回 oId」用例 - 文件头与函数注释同步口径
This commit is contained in:
parent
57debbe022
commit
b223db7961
@ -9,8 +9,9 @@
|
||||
* ② 不发 .xlsx 附件 —— 群机器人 webhook 发不了文件,业务也确认群里看文本就够。
|
||||
* 内容 = 汇总(发货/异常/异常率/分类型/分仓)+ 前 N 条明细 + 页面链接。
|
||||
* N 由页面配置(默认 30,上限 50);要全部明细就点链接去页面导出 Excel。
|
||||
* 明细一行 = 仓|内部单号|快递|异常类型|待处理小时数。
|
||||
* 明细一行 = 仓|快递单号|快递|异常类型|待处理小时数。
|
||||
* ⚠️ **不带发货时间**(业务 2026-10-22 要求),别再加 sendTime 回去。
|
||||
* ⚠️ 第二段是**快递单号**不是内部单号(同 10-22 二次要求)。
|
||||
* ③ 消息里**必须带「异常」字样**:钉钉群机器人若用「自定义关键词」安全设置,
|
||||
* 关键词一般是「异常」,缺了会被拒(errcode 310000)。
|
||||
* ④ 一天只推一次:状态存 system_config,按「推送日(北京时间)」去重,
|
||||
@ -83,14 +84,16 @@ function formatHours(value) {
|
||||
}
|
||||
|
||||
/**
|
||||
* 明细一行:仓|内部单号|快递|异常类型|待处理小时数
|
||||
* ⚠️ 业务明确要求明细**不带发货时间**(20-22 反馈),别再加 sendTime 回去 ——
|
||||
* 明细一行:仓|快递单号|快递|异常类型|待处理小时数
|
||||
* ⚠️ 业务明确要求明细**不带发货时间**(10-22 反馈),别再加 sendTime 回去 ——
|
||||
* 仓库看单子只需知道「哪个单、什么异常、压了多久」,发货时间在页面上看。
|
||||
* ⚠️ 第二段用**快递单号**(`trackingNo` = 上游 `logistic_id`),不是内部单号 `oId`(10-22 二次要求)。
|
||||
* 「未揽收 / 无物流单」这类包裹本来就没有快递单号 → 退回内部单号,否则群里这条无法定位。
|
||||
*/
|
||||
function buildDetailLine(row, index) {
|
||||
const parts = [
|
||||
cleanCell(row?.warehouseName, '未知仓'),
|
||||
cleanCell(row?.oId, '无单号'),
|
||||
cleanCell(row?.trackingNo || row?.oId, '无单号'),
|
||||
cleanCell(row?.carrier, '未知快递'),
|
||||
cleanCell(row?.exceptionLabel, '异常'),
|
||||
formatHours(row?.pendingHours),
|
||||
|
||||
@ -118,10 +118,11 @@ test('群消息正文:抬头带「异常」关键词、数字与分类型、
|
||||
assert.equal(built.truncated, false);
|
||||
});
|
||||
|
||||
test('明细行:仓|内部单号|快递|异常类型|待处理(不带发货时间),字段里换行要压平', () => {
|
||||
test('明细行:仓|快递单号|快递|异常类型|待处理(不带发货时间),字段里换行要压平', () => {
|
||||
const line = job.buildDetailLine({
|
||||
warehouseName: '崔桥工厂仓',
|
||||
oId: '9845001',
|
||||
trackingNo: 'JDVA00123456789',
|
||||
carrier: '京东快递',
|
||||
exceptionLabel: '已揽收未中转',
|
||||
sendTime: '2026-10-21 16:00:00',
|
||||
@ -129,13 +130,20 @@ test('明细行:仓|内部单号|快递|异常类型|待处理(不
|
||||
lastLogisticInfo: '第一行\n第二行',
|
||||
}, 3);
|
||||
|
||||
assert.equal(line, '3. 崔桥工厂仓|9845001|京东快递|已揽收未中转|17.3h');
|
||||
assert.equal(line, '3. 崔桥工厂仓|JDVA00123456789|京东快递|已揽收未中转|17.3h');
|
||||
// 业务要求:第二段是快递单号,不是内部单号
|
||||
assert.ok(!line.includes('9845001'), '明细行不该再显示内部单号');
|
||||
// 业务要求:明细不带发货时间(哪怕传了 sendTime 也不能出现在消息里)
|
||||
assert.ok(!line.includes('10-21'), '明细行不该再带发货时间');
|
||||
assert.ok(!line.includes('16:00'), '明细行不该再带发货时间');
|
||||
|
||||
// 物流描述里的换行不能漏进消息(会把一条明细劈成两行,看着像两条异常)
|
||||
assert.equal(job.cleanCell('甲\n乙\t丙 丁'), '甲 乙 丙 丁');
|
||||
// 未揽收/无物流单没有快递单号 → 退回内部单号(否则群里这条无法定位)
|
||||
assert.equal(
|
||||
job.buildDetailLine({ oId: '9845002', carrier: '中通' }, 1),
|
||||
'1. 未知仓|9845002|中通|异常|—'
|
||||
);
|
||||
// 空值兜底:不能让消息里出现「undefined|null」
|
||||
assert.equal(
|
||||
job.buildDetailLine({}, 1),
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user