diff options
author | Harrison Healey <harrisonmhealey@gmail.com> | 2016-07-06 16:07:56 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-07-06 16:07:56 -0400 |
commit | b114062c1b8128fe650fce72454ba4f1fcbe5733 (patch) | |
tree | 0a51e9d1342bf89c435662c618d9aac5ba651b4b /webapp/components/admin_console/webhook_settings.jsx | |
parent | b1520d0b9458d1dab09d9e3e05dd7522fc28ba57 (diff) | |
download | chat-b114062c1b8128fe650fce72454ba4f1fcbe5733.tar.gz chat-b114062c1b8128fe650fce72454ba4f1fcbe5733.tar.bz2 chat-b114062c1b8128fe650fce72454ba4f1fcbe5733.zip |
PLT-3237 Update displayed config values in admin console after saving (#3506)
* Reloaded admin console data when settings are saved
* Fixed attempting to save an invalid config setting overwriting the stored config
Diffstat (limited to 'webapp/components/admin_console/webhook_settings.jsx')
-rw-r--r-- | webapp/components/admin_console/webhook_settings.jsx | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/webapp/components/admin_console/webhook_settings.jsx b/webapp/components/admin_console/webhook_settings.jsx index 6c2e2d16a..18a3ed7ad 100644 --- a/webapp/components/admin_console/webhook_settings.jsx +++ b/webapp/components/admin_console/webhook_settings.jsx @@ -15,15 +15,6 @@ export default class WebhookSettings extends AdminSettings { this.getConfigFromState = this.getConfigFromState.bind(this); this.renderSettings = this.renderSettings.bind(this); - - this.state = Object.assign(this.state, { - enableIncomingWebhooks: props.config.ServiceSettings.EnableIncomingWebhooks, - enableOutgoingWebhooks: props.config.ServiceSettings.EnableOutgoingWebhooks, - enableCommands: props.config.ServiceSettings.EnableCommands, - enableOnlyAdminIntegrations: props.config.ServiceSettings.EnableOnlyAdminIntegrations, - enablePostUsernameOverride: props.config.ServiceSettings.EnablePostUsernameOverride, - enablePostIconOverride: props.config.ServiceSettings.EnablePostIconOverride - }); } getConfigFromState(config) { @@ -37,6 +28,17 @@ export default class WebhookSettings extends AdminSettings { return config; } + getStateFromConfig(config) { + return { + enableIncomingWebhooks: config.ServiceSettings.EnableIncomingWebhooks, + enableOutgoingWebhooks: config.ServiceSettings.EnableOutgoingWebhooks, + enableCommands: config.ServiceSettings.EnableCommands, + enableOnlyAdminIntegrations: config.ServiceSettings.EnableOnlyAdminIntegrations, + enablePostUsernameOverride: config.ServiceSettings.EnablePostUsernameOverride, + enablePostIconOverride: config.ServiceSettings.EnablePostIconOverride + }; + } + renderTitle() { return ( <h3> |