From e60fb1dfd77609c21689dc5d3d0ba001229fbed4 Mon Sep 17 00:00:00 2001 From: Christopher Speller Date: Tue, 13 Oct 2015 11:31:57 -0400 Subject: Fixing user settings dialog so it desn't ask you to check your email unless you changed it --- .../user_settings/user_settings_general.jsx | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) (limited to 'web/react/components/user_settings') diff --git a/web/react/components/user_settings/user_settings_general.jsx b/web/react/components/user_settings/user_settings_general.jsx index c23c61948..ac3a2e37b 100644 --- a/web/react/components/user_settings/user_settings_general.jsx +++ b/web/react/components/user_settings/user_settings_general.jsx @@ -59,7 +59,7 @@ export default class UserSettingsGeneralTab extends React.Component { user.username = username; - this.submitUser(user); + this.submitUser(user, false); } submitNickname(e) { e.preventDefault(); @@ -74,7 +74,7 @@ export default class UserSettingsGeneralTab extends React.Component { user.nickname = nickname; - this.submitUser(user); + this.submitUser(user, false); } submitName(e) { e.preventDefault(); @@ -91,7 +91,7 @@ export default class UserSettingsGeneralTab extends React.Component { user.first_name = firstName; user.last_name = lastName; - this.submitUser(user); + this.submitUser(user, false); } submitEmail(e) { e.preventDefault(); @@ -115,22 +115,22 @@ export default class UserSettingsGeneralTab extends React.Component { } user.email = email; - this.submitUser(user); + this.submitUser(user, true); } - submitUser(user) { + submitUser(user, emailUpdated) { client.updateUser(user, - function updateSuccess() { + () => { this.updateSection(''); AsyncClient.getMe(); - const verificationEnabled = global.window.config.SendEmailNotifications === 'true' && global.window.config.RequireEmailVerification === 'true'; + const verificationEnabled = global.window.config.SendEmailNotifications === 'true' && global.window.config.RequireEmailVerification === 'true' && emailUpdated; if (verificationEnabled) { ErrorStore.storeLastError({message: 'Check your email at ' + user.email + ' to verify the address.'}); ErrorStore.emitChange(); this.setState({emailChangeInProgress: true}); } - }.bind(this), - function updateFailure(err) { + }, + (err) => { var state = this.setupInitialState(this.props); if (err.message) { state.serverError = err.message; @@ -138,7 +138,7 @@ export default class UserSettingsGeneralTab extends React.Component { state.serverError = err; } this.setState(state); - }.bind(this) + } ); } submitPicture(e) { -- cgit v1.2.3-1-g7c22 From ac0f8f6bcf9eb65b2541d537ef8960f0b5d3f0ff Mon Sep 17 00:00:00 2001 From: JoramWilander Date: Tue, 13 Oct 2015 18:01:11 -0400 Subject: Fix webhook name not overriding in LHS and update webhook help text. --- web/react/components/user_settings/manage_incoming_hooks.jsx | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'web/react/components/user_settings') diff --git a/web/react/components/user_settings/manage_incoming_hooks.jsx b/web/react/components/user_settings/manage_incoming_hooks.jsx index 8817e0d39..f5a2774a0 100644 --- a/web/react/components/user_settings/manage_incoming_hooks.jsx +++ b/web/react/components/user_settings/manage_incoming_hooks.jsx @@ -148,7 +148,8 @@ export default class ManageIncomingHooks extends React.Component { return (
- {'For developers building integrations this page lets you create webhook URLs for channels and private groups. Please see http://mattermost.org/webhooks to learn about creating webhooks, view samples, and to let the community know about integrations you have built. The URLs created below can be used by outside applications to create posts in any channels or private groups you have access to. The specified channel will be used as the default.'} + {'Create webhook URLs for use in external integrations. Please see '}{'http://mattermost.org/webhooks'} {' to learn more.'} +

-- cgit v1.2.3-1-g7c22