Skip to content

feat: expand SQL manage export columns with rule_name - #892

Merged
Seechi-Yolo merged 5 commits into
zjrc_3.2408from
dms-ui/feat-3127
Sep 2, 2026
Merged

feat: expand SQL manage export columns with rule_name#892
Seechi-Yolo merged 5 commits into
zjrc_3.2408from
dms-ui/feat-3127

Conversation

@LordofAvernus

Copy link
Copy Markdown
Collaborator

关联的 issue

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

描述你的变更

  • SQL 管控列表导出列键将 audit_result 展开为 audit_level,rule_name,rule_desc
  • 稳定 AuditResultList 分页单测 mock,避免 CI 超时
  • 同步当前分支 UI_VERSION

确认项(pr提交后操作)

Tip

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


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

Made with Cursor

LordofAvernus and others added 3 commits September 1, 2026 18:27
Map list audit_result to audit_level, rule_name, rule_desc for
export_column_keys. Relates to actiontech/sqle-ee#3127

Co-authored-by: Cursor <cursoragent@cursor.com>
Slice mock data by page and cap rows to keep the suite within timeout.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 90.41% 14965/16553
🟡 Branches 73.23% 6089/8315
🟢 Functions 90.61% 5298/5847
🟢 Lines 90.9% 14702/16173

Test suite run success

2313 tests passing in 629 suites.

Report generated by 🧪jest coverage report action from 7ad2d75

LordofAvernus and others added 2 commits September 2, 2026 12:26
Keep pagination coverage with full mock data and CI timeout guard while
removing rule_template setup that is unrelated to SQL manage export work.

Co-authored-by: Cursor <cursoragent@cursor.com>
Refresh VersionModal snapshots after branch tip moves past review cleanup.

Co-authored-by: Cursor <cursoragent@cursor.com>
@Seechi-Yolo
Seechi-Yolo merged commit 8b55f5d into zjrc_3.2408 Sep 2, 2026
9 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