-
Notifications
You must be signed in to change notification settings - Fork 614
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
sysutils/monit: bug fixing, status api response #563
Conversation
fbrendel
commented
Feb 15, 2018
- SettingsController: fix faulty deletion of alerts, cleanup code, new function notificationAction to import system notification settings
- ServiceController: is a ApiMutableServiceControllerBase now
- StatusController: retruns the Monit status as JSON, old behaviour via 'html' parameter
- UI: accept FQDN as mailserver, 'Apply' doesn't restart Monit
@@ -1,7 +1,7 @@ | |||
<?php | |||
|
|||
/** | |||
* Copyright (C) 2017 EURO-LOG AG | |||
* Copyright (C) 2017, 2018 EURO-LOG AG |
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.
2017-2018
Looks good, but we should get the post-install hook out with a native config migration eventually. How much do you plan to edit this in the next 1-2 months? If you say not a lot we should merge it to core in March as discussed previously. |
Not much. Small bug fixes eventually. |
Merged, thanks! Ok, then it's a plan... :) |
* can't do it via base_dialog | ||
*/ | ||
$('<button class="btn btn-primary" id="btn_ImportSystemNotification" type="button" style="margin-left: 3px;">' + | ||
'<b> Import System Notification </b>' + |
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.
this line is missing translation