-
Notifications
You must be signed in to change notification settings - Fork 994
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
Alert yml config #1535
Alert yml config #1535
Conversation
👍,hi建议不把不同环境分离出来做成独立yml,因为docker挂载的时候就需要再挂载一个,但是我们对外文档说明一直都是挂载那个 application.yml,改了之后会造成疑虑 |
👍, Hi, it is not recommended to separate different environments into independent ymls, because when docker is mounted, another one needs to be mounted. However, our external documentation always states that the application.yml is mounted. Changing it will cause doubts. |
alerter/src/main/java/org/dromara/hertzbeat/alert/AlerterProperties.java
Outdated
Show resolved
Hide resolved
ok |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!👍👍
Co-authored-by: tomsun28 <tomsun28@outlook.com>
Co-authored-by: tomsun28 <tomsun28@outlook.com>
Co-authored-by: tomsun28 <tomsun28@outlook.com>
Co-authored-by: tomsun28 <tomsun28@outlook.com>
Co-authored-by: tomsun28 <tomsun28@outlook.com>
Co-authored-by: tomsun28 <tomsun28@outlook.com>
Co-authored-by: tomsun28 <tomsun28@outlook.com>
Co-authored-by: tomsun28 <tomsun28@outlook.com>
Co-authored-by: tomsun28 <tomsun28@outlook.com>
What's changed?
Alert yml config
Checklist
Add or update API