Skip to content

feat: strengthen environment tag color display (#2983)#3340

Merged
LordofAvernus merged 1 commit into
mainfrom
sqle/feat-2983
Jun 22, 2026
Merged

feat: strengthen environment tag color display (#2983)#3340
LordofAvernus merged 1 commit into
mainfrom
sqle/feat-2983

Conversation

@Seechi-Yolo

@Seechi-Yolo Seechi-Yolo commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator

User description

关联的 issue

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

描述你的变更

在 SQLE 数据源 tip 接口中补充环境标签颜色字段,使 SQLE 侧选源下拉可与 DMS 侧一致展示带色环境标签。

主要改动

  • getInstanceTipListV2:在已有 environment_tag_name 基础上新增 environment_tag_color
  • 同步更新 controller、model、dms 调用层及 swagger 文档
  • vendor:同步 dms-commonListDBServiceTipItem 类型(随 dms MR 合并后对齐)

兼容性

  • 环境标签无 color 或数据源未打标时,字段为空,前端降级为中性灰或不展示标签,行为与现网一致
  • 选择生产环境数据源 不弹窗、不阻断

依赖关系

  • 建议在 dms MR 合并后 再合并本 MR(vendor 引用 dms-common 类型)

确认项(pr提交后操作)

Tip

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


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


Description

  • 新增环境标签颜色字段

  • 更新API返回数据结构

  • 调整DMS数据转换逻辑

  • 修订文档与swagger描述


Diagram Walkthrough

flowchart LR
  A["API Controller更新"] -- "新增环境标签字段" --> B["模型字段增强"]
  A -- "传递环境标签数据" --> C["DMS转换逻辑"]
  D["文档更新"] -- "配置swagger更新" --> A
Loading

File Walkthrough

Relevant files
Enhancement
4 files
instance.go
为实例响应添加环境标签颜色字段                                                                                   
+18/-12 
instance_audit_plan.go
为实例提示响应新增环境标签颜色字段                                                                               
+2/-0     
instance.go
更新实例转换逻辑加入环境标签颜色                                                                                 
+21/-19 
instance.go
模型中增加环境标签颜色字段                                                                                       
+3/-2     
Documentation
3 files
docs.go
修改文档和swagger,添加环境标签颜色及assignee字段                                                 
+19/-0   
swagger.json
swagger文档中新增环境标签颜色定义                                                                         
+19/-0   
swagger.yaml
swagger YAML文件中新增环境标签颜色描述                                                               
+13/-0   

@github-actions

Copy link
Copy Markdown

PR Reviewer Guide 🔍

🎫 Ticket compliance analysis 🔶

2983 - Partially compliant

Compliant requirements:

  • 完成自测
  • 记录完整日志方便进行诊断
  • 在关联的 issue 中补充实现方案
  • 在关联的 issue 中补充测试影响面
  • 确认变更的兼容性
  • 确认是否需要更新文档

Non-compliant requirements:

无不符合项

Requires further human verification:

无需额外人工验证项

⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪
🧪 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 c88e0b4 into main Jun 22, 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