Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: 通知方式分组 支持将不同的报警|监控|计划任务的通知 发送到指定的通知分组 #160

Merged
merged 9 commits into from
Apr 16, 2022

Conversation

AkkiaS7
Copy link
Member

@AkkiaS7 AkkiaS7 commented Apr 14, 2022

增加为通知方式设置分组的功能
为各项 报警|监控|计划任务 设置不同的通知方式分组
事件发生时将通过其对应的通知方式分组发送消息

为了兼容之前的版本
后端运行初始化过程中会将所有为指定通知组的相关对象指定至默认通知组default
在前端编辑相关对象时 会判断是否添加了通知组 未添加则默认指定为default

  • 为通知方式提供分组功能
  • 为报警增加通知分组
  • 为计划任务增加通知分组
  • 为服务监控增加通知分组
  • 前端修改
  • 检查前向兼容性
  • 功能测试

工作完成

@AkkiaS7 AkkiaS7 changed the title WIP: 通知方式分组 支持将不同的报警|监控|计划任务的通知 发送到指定的通知分组 feat: 通知方式分组 支持将不同的报警|监控|计划任务的通知 发送到指定的通知分组 Apr 15, 2022
@AkkiaS7
Copy link
Member Author

AkkiaS7 commented Apr 15, 2022

功能验证:
7dc014402a35d734356fc962cbecc06a.png
c8205c3ed2b5a65a5b6dbfa059637449.png
5a5133915dae783a194c8985056febcc.png
32d9809b16a7c42ed2a72b6b1eac36a8.png
f8521c793029dbc6587d36588c1d5d6b.png
25e5295d98e4169db5bc0c786174b9af.png

model/config.go Outdated Show resolved Hide resolved
Copy link
Member

@naiba naiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Request Changes

@AkkiaS7
Copy link
Member Author

AkkiaS7 commented Apr 15, 2022

修复后的测试
09ca1d52cb700ecd74eeac7c43bd929a.png

@AkkiaS7 AkkiaS7 requested a review from naiba April 15, 2022 15:26
Co-authored-by: AkkiaS7 <[email protected]>
@naiba naiba merged commit 61baa31 into nezhahq:master Apr 16, 2022
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