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.

Sign up for GitHub

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

Jump to bottom

[improve] update monitor state desc #2028

Merged
merged 4 commits into from
May 23, 2024
Merged

[improve] update monitor state desc #2028

merged 4 commits into from
May 23, 2024

Conversation

tomsun28
Copy link
Contributor

What's changed?

  • update monitor state desc
  • update monitor list button

Checklist

  • I have read the Contributing Guide
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

Add or update API

  • I have added the necessary e2e tests and all cases have passed.

@tomsun28
update monitor state desc
6c6d5cf 
Signed-off-by: tomsun28 <tomsun28@outlook.com>
@tomsun28
update monitor status desc
708bde9 
Signed-off-by: tomsun28 <tomsun28@outlook.com>
@tomsun28 tomsun28 requested a review from TJxiaobao May 23, 2024 09:27
@tomsun28 tomsun28 added this to the 1.6.0 milestone May 23, 2024
@tomsun28
update monitor state desc
1629d9a 
Signed-off-by: tomsun28 <tomsun28@outlook.com>
@tomsun28
update monitor status desc
8f038c2 
Signed-off-by: tomsun28 <tomsun28@outlook.com>
TJxiaobao
TJxiaobao approved these changes May 23, 2024
View reviewed changes
Copy link
Contributor

@TJxiaobao TJxiaobao left a comment

Choose a reason for hiding this comment

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

👍👍👍

@TJxiaobao TJxiaobao merged commit a78f71d into master May 23, 2024
4 checks passed
@zqr10159 zqr10159 deleted the update-monitor-state branch September 23, 2024 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TJxiaobao TJxiaobao TJxiaobao approved these changes

Assignees
No one assigned
Labels
backend webapp
Projects
Apache HertzBeat (Incubating)
Status: Done
Milestone
1.6.0
Development

Successfully merging this pull request may close these issues.
2 participants
@tomsun28 @TJxiaobao