[GitHub] [skywalking] wu-sheng commented on a change in pull request #6193: Support Alarm to feishu

[GitBox <gi...@apache.org>]

wu-sheng commented on a change in pull request #6193:
URL: https://github.com/apache/skywalking/pull/6193#discussion_r557016608



##########
File path: docs/en/setup/backend/backend-alarm.md
##########
@@ -237,11 +237,31 @@ dingtalkHooks:
       secret: dummysecret
 ```
 
-
 ## Update the settings dynamically
 Since 6.5.0, the alarm settings can be updated dynamically at runtime by [Dynamic Configuration](dynamic-config.md),
 which will override the settings in `alarm-settings.yml`.
 
 In order to determine that whether an alarm rule is triggered or not, SkyWalking needs to cache the metrics of a time window for
 each alarm rule, if any attribute (`metrics-name`, `op`, `threshold`, `period`, `count`, etc.) of a rule is changed,
 the sliding window will be destroyed and re-created, causing the alarm of this specific rule to restart again.
+
+
+## Feishu Hook

Review comment:
       Move this before `Update the settings dynamically` part.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org