From e700a5a21942ff7b95434a083dedd4c49690e823 Mon Sep 17 00:00:00 2001 From: Kegan Dougal Date: Thu, 19 Nov 2015 13:58:52 +0000 Subject: [PATCH] Add TODO on post register logic --- src/components/login/Registration.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/components/login/Registration.js b/src/components/login/Registration.js index 38cc51b7..627f9b27 100644 --- a/src/components/login/Registration.js +++ b/src/components/login/Registration.js @@ -86,7 +86,6 @@ module.exports = React.createClass({ }); return; } - // TODO: do post-register stuff self.props.onLoggedIn({ userId: response.user_id, homeserverUrl: self.props.registerLogic.getHomeserverUrl(), @@ -129,7 +128,10 @@ module.exports = React.createClass({ this.props.registerLogic.tellStage("m.login.recaptcha", "loaded"); }, - // TODO: I wonder if this should actually be a different component... + // TODO: + // This should really be a different component which MatrixChat then + // instantiates rather than having it pollute registration logic. There is + // no reason to wedge them together here. This function is currently NOT CALLED. _getPostRegisterJsx: function() { var ChangeDisplayName = sdk.getComponent('molecules.ChangeDisplayName'); var ChangeAvatar = sdk.getComponent('molecules.ChangeAvatar');