Skip to content

chore(workflow): add filter status to global workflow list request fo…#3311

Merged
LordofAvernus merged 1 commit into
mainfrom
dsb_add_parameters
May 21, 2026
Merged

chore(workflow): add filter status to global workflow list request fo…#3311
LordofAvernus merged 1 commit into
mainfrom
dsb_add_parameters

Conversation

@iwanghc
Copy link
Copy Markdown
Collaborator

@iwanghc iwanghc commented May 21, 2026

User description

…r enhanced management

关联的 issue

https://github.com/actiontech/sqle-ee/issues/2884

描述你的变更

全局dashboard工单查询参数

确认项(pr提交后操作)

Tip

请在指定复审人之前,确认并完成以下事项,完成后✅


  • 我已完成自测
  • 我已记录完整日志方便进行诊断
  • 我已在关联的issue里补充了实现方案
  • 我已在关联的issue里补充了测试影响面
  • 我已确认了变更的兼容性,如果不兼容则在issue里标记 not_compatible
  • 我已确认了是否要更新文档,如果要更新则在issue里标记 need_update_doc

link #3309


Description

  • 新增 FilterStatus 字段用于过滤工单状态

  • 支持统一状态枚举值进行校验

  • 增强全局工作流列表的筛选功能


Diagram Walkthrough

flowchart LR
  A["新增字段 FilterStatus"] -- "添加至" --> B["全局工作流请求结构体"]
  B -- "实现统一过滤" --> C["筛选结果优化"]
Loading

File Walkthrough

Relevant files
Enhancement
workflow_manage.go
新增工单状态过滤字段                                                                                             

sqle/api/controller/v2/dashboard/workflow_manage.go

  • 在 GetGlobalWorkflowListReqV2 结构体新增 FilterStatus 字段
  • 定义字段类型为 dashboard_svc.UnifiedWorkflowStatus
  • 增加字段的 json, query 标记及校验规则
+3/-0     

@actiontech-bot actiontech-bot requested review from LordofAvernus and removed request for LordofAvernus May 21, 2026 08:45
@github-actions
Copy link
Copy Markdown

PR Reviewer Guide 🔍

🎫 Ticket compliance analysis 🔶

2884 - Partially compliant

Compliant requirements:

  • 增加了 FilterStatus 字段用于过滤工单状态,支持统一状态枚举值校验

Non-compliant requirements:

Requires further human verification:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions
Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

@LordofAvernus LordofAvernus merged commit 01d486e into main May 21, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants