|
|
@@ -70,3 +70,58 @@
|
|
|
|
|
|
- Issue #8 尚未在 Gogs 关闭，等待用户人工验收。
|
|
|
- KingbaseES 已完成测试数据脚本验证；页面级人工验收仍建议在真实服务与浏览器中确认导出文件和趋势交互。
|
|
|
+
|
|
|
+## 2026-09-01：供水运行监测 T4 第四轮修正
|
|
|
+
|
|
|
+### 结论
|
|
|
+
|
|
|
+- 流量分页在雷达与遥测数据同时存在时，改为按采集时间选择真正最新的一条测量值，不再固定优先雷达数据。
|
|
|
+- 四类趋势查询在只传结束时间时不再绑定 `NULL` 起点条件，避免 KingbaseES 中 `create_time >= NULL` 导致查询为空。
|
|
|
+- 前端恢复 `Refresh` 图标导入，保证供水监测页刷新按钮正常渲染。
|
|
|
+- 新增后端回归测试覆盖跨数据源最新流量值和仅结束时间的趋势查询参数绑定。
|
|
|
+
|
|
|
+### 验证
|
|
|
+
|
|
|
+- 后端 `mvn -q -pl zksy-system -Dtest=MonitoringServiceImplTest test` 通过。
|
|
|
+- 后端 `mvn -q test` 通过。
|
|
|
+- 前端 `node --test tests/waterSupplyMonitoring.test.mjs` 1/1 通过。
|
|
|
+- 前端 `npm run build:prod` 通过，仅存在既有 Sass 弃用警告。
|
|
|
+
|
|
|
+### 风险与后续
|
|
|
+
|
|
|
+- Issue #8 继续保留给用户人工验收；本轮未连接真实 KingbaseES 页面环境复测浏览器交互。
|
|
|
+
|
|
|
+## 2026-09-01：供水运行监测 T4 标准审查
|
|
|
+
|
|
|
+### 结论
|
|
|
+
|
|
|
+- 按 `origin/master...HEAD` 审查前端 2 个提交、后端 3 个提交；确认供水设备树过滤、共享监测页、右侧详情抽屉、显式状态、权限导出和测试覆盖基本符合项目标准。
|
|
|
+- 发现 1 个硬性标准问题：监测列表与趋势请求失败时先清空既有数据，违反供水工单 UI 设计中“接口失败在原位置显示错误与重试入口，保留已有数据和表单内容”的要求。
|
|
|
+- 另记录趋势接口 Swagger 文案仍写“近24小时”、全量内存分页与导出、控制器/基础字段填充重复、运行时类型选择器等 judgement-call 风险。
|
|
|
+
|
|
|
+### 验证
|
|
|
+
|
|
|
+- 本轮为只读标准审查，未修改业务代码，未重新运行测试；沿用本轮审查证据和既有会话记录中的验证结果。
|
|
|
+
|
|
|
+### 后续行动
|
|
|
+
|
|
|
+- 修复失败请求时的既有数据保留逻辑；建议同步修正趋势接口文案，并在数据量扩大前评估数据库分页与流式导出。
|
|
|
+
|
|
|
+## 2026-09-01：供水运行监测 T4 验收收尾
|
|
|
+
|
|
|
+### 结论
|
|
|
+
|
|
|
+- 修复监测列表请求失败时清空旧数据的问题；失败时保留原列表、分页和筛选上下文，仅显示错误与重试入口。
|
|
|
+- 趋势请求重试失败时保留当前设备旧图表；切换到新设备时才重置趋势上下文，避免旧设备数据误导当前详情。
|
|
|
+- 双轴审查结论：Spec 轴未见阻塞性缺口；Standards 轴原硬性问题已修复，剩余 Swagger 文案、内存分页/导出与重复代码为非阻塞改进项。
|
|
|
+
|
|
|
+### 验证
|
|
|
+
|
|
|
+- 前端 `node --test tests/waterSupplyMonitoring.test.mjs` 通过。
|
|
|
+- 前端 `npm run build:prod` 通过，仅存在既有 Sass 弃用警告。
|
|
|
+- 后端本轮未再改动，沿用本轮 `mvn -q test` 通过结果。
|
|
|
+
|
|
|
+### 风险与后续
|
|
|
+
|
|
|
+- Issue #8 仍保持开启，等待用户在 KingbaseES 与真实浏览器中完成人工验收。
|
|
|
+- 建议后续单独处理趋势接口 Swagger 文案、数据库侧分页/流式导出和三类监测控制器重复逻辑。
|
