Skip to content

Conversation

@BugsGuru
Copy link
Collaborator

@BugsGuru BugsGuru commented Oct 27, 2025

User description

关联的 issue

#3159

描述你的变更

美化通知内容

确认项(pr提交后操作)

Tip

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


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


Description

  • 使用 emoji 美化通知内容

  • 调整工单通知文本排版

  • 优化通知模板和分隔线样式


Diagram Walkthrough

flowchart LR
  A["更新通知格式"] --> B["更新 locale 消息"]
  A --> C["更新 notification 功能"]
  A --> D["更新 active 模板文件"]
Loading

File Walkthrough

Relevant files
Enhancement
message_zh.go
更新中文工单通知格式                                                                                             

sqle/locale/message_zh.go

  • 修改工单通知文本使用 emoji
  • 调整文本格式与分隔符样式
  • 更新错误提示和评分显示信息
+8/-8     
notification.go
优化通知方法排版                                                                                                 

sqle/notification/notification.go

  • 添加空行提高可读性
  • 修改分隔符为横向线条
+4/-1     
active.en.toml
更新英文通知模板格式                                                                                             

sqle/locale/active.en.toml

  • 更新英文通知模板,加入 emoji
  • 调整通知文本格式
+8/-8     
active.zh.toml
更新中文通知模板格式                                                                                             

sqle/locale/active.zh.toml

  • 更新中文通知模板,文本美化
  • 替换标识符和分隔线样式
+8/-8     

@github-actions
Copy link

github-actions bot commented Oct 27, 2025

PR Reviewer Guide 🔍

(Review updated until commit 654cea3)

🎫 Ticket compliance analysis 🔶

3159 - Partially compliant

Compliant requirements:

  • 美化通知内容
  • 调整工单通知文本排版
  • 优化通知模板和分隔线样式

Non-compliant requirements:

Requires further human verification:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

格式更新

请检查本地化消息中 emoji 与文案之间的布局和标点符号是否与现有项目规范保持一致,确保多语言版本的一致性。

	NotifyWorkflowBodyHead              = &i18n.Message{ID: "NotifyWorkflowBodyHead", Other: "\n📋 工单主题: %v\n📍 所属项目: %v\n🆔 工单ID: %v\n📝 工单描述: %v\n👤 申请人: %v\n⏰ 创建时间: %v\n"}
	NotifyWorkflowBodyWorkFlowErr       = &i18n.Message{ID: "NotifyWorkflowBodyWorkFlowErr", Other: "❌ 读取工单任务内容失败,请通过SQLE界面确认工单状态"}
	NotifyWorkflowBodyLink              = &i18n.Message{ID: "NotifyWorkflowBodyLink", Other: "🔗 工单链接: %v"}
	NotifyWorkflowBodyConfigUrl         = &i18n.Message{ID: "NotifyWorkflowBodyConfigUrl", Other: "请在系统设置-全局配置中补充全局url"}
	NotifyWorkflowBodyInstanceErr       = &i18n.Message{ID: "NotifyWorkflowBodyInstanceErr", Other: "❌ 获取数据源实例失败: %v\n"}
	NotifyWorkflowBodyInstanceAndSchema = &i18n.Message{ID: "NotifyWorkflowBodyInstanceAndSchema", Other: "🗄️ 数据源: %v\n📊 Schema: %v\n"}
	NotifyWorkflowBodyStartEnd          = &i18n.Message{ID: "NotifyWorkflowBodyStartEnd", Other: "▶️ 上线开始时间: %v\n◀️ 上线结束时间: %v\n"}
	NotifyWorkflowBodyReason            = &i18n.Message{ID: "NotifyWorkflowBodyReason", Other: "❌ 驳回原因: %v\n"}
	NotifyWorkflowBodyReport            = &i18n.Message{ID: "NotifyWorkflowBodyReport", Other: "✅ 工单审核得分: %v\n"}
)
可读性优化

新增的空行和分隔线更新有助于提高文本可读性,请确认该改动不会造成通知内容解析问题。

// Add a blank line before task details for better readability
bodyStr = append(bodyStr, i18nPkg.ConvertStr2I18nAsDefaultLang(""))

for _, t := range tasks {
	if instance, ok := instanceMap[t.InstanceId]; ok {
		t.Instance = instance
	}

	bodyStr = append(bodyStr, i18nPkg.ConvertStr2I18nAsDefaultLang("────────────────"), w.buildNotifyBody(t))
}

@github-actions
Copy link

PR Code Suggestions ✨

No code suggestions found for the PR.

@BugsGuru BugsGuru force-pushed the update-notification branch from 0b3ad91 to 654cea3 Compare October 27, 2025 09:49
@github-actions
Copy link

Persistent review updated to latest commit 654cea3

@github-actions
Copy link

PR Code Suggestions ✨

No code suggestions found for the PR.

@LordofAvernus LordofAvernus merged commit 179a3b9 into main Oct 31, 2025
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.

3 participants