From 7ae1ff439df66902e62dd181a9e827cc82dd6e26 Mon Sep 17 00:00:00 2001 From: Neil Lalonde Date: Mon, 22 Jun 2015 15:36:42 -0400 Subject: [PATCH] FIX: modal explaining that your account hasn't been activated yet was broken if you saw it more than once --- app/assets/javascripts/discourse/controllers/login.js.es6 | 3 ++- .../javascripts/discourse/controllers/not-activated.js.es6 | 4 ++++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/app/assets/javascripts/discourse/controllers/login.js.es6 b/app/assets/javascripts/discourse/controllers/login.js.es6 index 7db0f9b3aa..f5b31411f9 100644 --- a/app/assets/javascripts/discourse/controllers/login.js.es6 +++ b/app/assets/javascripts/discourse/controllers/login.js.es6 @@ -69,8 +69,9 @@ export default DiscourseController.extend(ModalFunctionality, { sentTo: result.sent_to_email, currentEmail: result.current_email }); + } else { + self.flash(result.error, 'error'); } - self.flash(result.error, 'error'); } else { self.set('loggedIn', true); // Trigger the browser's password manager using the hidden static login form: diff --git a/app/assets/javascripts/discourse/controllers/not-activated.js.es6 b/app/assets/javascripts/discourse/controllers/not-activated.js.es6 index 3ce160c880..dd53320766 100644 --- a/app/assets/javascripts/discourse/controllers/not-activated.js.es6 +++ b/app/assets/javascripts/discourse/controllers/not-activated.js.es6 @@ -4,6 +4,10 @@ import DiscourseController from 'discourse/controllers/controller'; export default DiscourseController.extend(ModalFunctionality, { emailSent: false, + onShow() { + this.set("emailSent", false); + }, + actions: { sendActivationEmail: function() { Discourse.ajax('/users/action/send_activation_email', {data: {username: this.get('username')}, type: 'POST'});