Repository navigation
fix(v2): keep inspection review, task results, and workflow stages consistent - #352
Merged
wanglongan587 merged 12 commits intoNov 9, 2026
Merged
Conversation
wanglongan587
force-pushed
the
fix/1012-daily-inspection-ui
branch
2 times, most recently
from
November 6, 2026 07:47
7b4249e to
4f84537
Compare
wanglongan587
force-pushed
the
fix/1012-daily-inspection-ui
branch
from
November 6, 2026 07:58
4f84537 to
63925d8
Compare
- 一键判定每项发现(是问题/不是问题),并把确认人预填为负责人,替代藏在原生下拉里的第三种状态 - 支持批量标记与撤销;底栏明确写出提交后会新建几个、关联几个任务 - 报告收进左栏、窄屏移到列表之后,Markdown 列表与表格恢复可读 - 提交后按结论阅读:不再渲染禁用的输入框,判为不是问题的条目说明不会建单 - 没有发现问题时只需一次确认;表单节点失败时说明原因,而不是让人反复刷新 - 任务结果页按冻结的发现补上标题与严重级别,失败时回到确认节点重试
- 删掉空操作的「应用表单配置」按钮;配置改动本来就是即时的 - 给每个附加字段标注标识、显示名、类型和必填,替换只有 aria-label 的裸输入框 - 说明报告/问题列表来源变量各需要什么形状,选中错误不必等到运行才暴露
复用已有任务的条目也显示“负责人(沿用) <姓名>”,新建任务的条目才给下拉框; 之前只有“会被新建”的那一行出现负责人控件,看起来像别的行漏了字段。 - 负责人槽位对每一行都在同一位置出现:沿用任务只读,新建任务可选 - 任务关联信息不再重复负责人,改为“关联到已有任务 #id · 状态 · 查看任务” - 去掉关联标签的底色,避免看着像可点击的按钮
复用任务的行也给出同一个负责人下拉框,默认带出该任务的负责人;之前只有 “会新建任务”的行有控件,复用的行是只读文字,同一张页面里看起来像漏字段。 - 判定为问题后每行都有负责人,位置与宽度一致 - 复用任务仍显示“关联到已有任务 #id · 状态 · 查看任务”,不再重复负责人 - 结论态继续按文本展示负责人,不渲染禁用的控件
- 失败在确认节点的运行直接落在表单节点上,先看到“确认表单没有生成”的原因,而不是一张通用失败页 - 非确认人看到只读说明位于标题下方,底部不再渲染一整套禁用按钮;运行状态改回中性的“等待处理”,不再对只读成员说“待你处理”
十个场景跑在真实 v2-server + PostgreSQL 上:判定/批量/撤销/筛选、草稿与刷新、 建单与任务页回链、跨天复用、无问题、只读成员、等待中取消、390px、失败绑定、 双人并发冲突、30 条长列表、每段节点各自视图、定时任务停在表单并可再次触发。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
每日巡检的确认表单曾替换所有节点视图,判定、批量操作、任务回执与页面状态也存在不一致。本 PR 保留各节点自己的内容,并让报告确认、任务创建/关联与结果查看形成连续的前端路径。
c00f53d0修复本地验收发现的三项问题:运行已成功但表单回执未到齐时显示“正在同步任务结果”,不误报“无需处理”;载入最新草稿时清除旧冲突错误、校验和撤销记录;表单主体独立滚动,页脚保留布局空间。frontend-v2/;v2,不涉及 specs。没有修改后端、接口、数据库或外部系统写入语义。Refs ora-space/circus#1012
Evidence
验证提交:
c00f53d0b51afced9187e4a52ded46b272c6dfaa。After:
run-form.test.tsx17 个测试通过;实际浏览器两标签页冲突恢复、任务回执自动更新,以及 1280×800 / 390×844 按钮点击与页脚布局均通过。git diff --check通过。workflow-review.spec.ts/workflow-forms.spec.ts共 13 个相关 e2e 用例均已验证通过,覆盖草稿、批量/筛选、权限、跨次任务复用、取消、窄屏、节点视图、并发冲突、长列表、定时记录、页脚布局和延迟回执。首轮末项遇到并行 Playwright 运行共用产物目录的 ENOENT,使用独立输出目录复跑通过。最新浏览器截图均来自
c00f53d0,存放在独立screenshots/1012分支,固定到截图提交e42e38e1,不进入产品 diff。截图使用本地 fixture 数据;任务保存/创建/关联为真实本地后端行为,DTS 回执仅演示,未写入外部 DTS;本次未执行实时 Agent 采集。Merge Danger
Door: two-way。纯前端变更,可回退;无数据迁移。
Blast Radius: 巡检界面。影响表单与任务节点的展示、交互及滚动布局;其他节点视图、已有任务负责人和后端建单语义保持原有规则。表单等待仍无截止时间、待办聚合或通知;真实 DTS 集成不在本 PR 范围内。