From 49545ce0c2802b0f48b9539edc9ef1a279c37817 Mon Sep 17 00:00:00 2001 From: wmwragg Date: Thu, 28 Jul 2016 15:29:57 +0100 Subject: [PATCH] Leaving the fail promise section blank as a todo, with explanitary comment --- .../views/context_menus/NotificationStateContextMenu.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/components/views/context_menus/NotificationStateContextMenu.js b/src/components/views/context_menus/NotificationStateContextMenu.js index 8f3152d2..879234fa 100644 --- a/src/components/views/context_menus/NotificationStateContextMenu.js +++ b/src/components/views/context_menus/NotificationStateContextMenu.js @@ -56,7 +56,6 @@ module.exports = React.createClass({ cli.setRoomMutePushRule( "global", roomId, isMuted ).then(function() { - console.log("DEBUG: then"); self.setState({areNotifsMuted: isMuted}); // delay slightly so that the user can see their state change @@ -66,8 +65,8 @@ module.exports = React.createClass({ }; }); }).fail(function(error) { - console.log("DEBUG: fail"); - console.log(error); + // TODO: some form of error notification to the user + // to inform them that their state change failed. }); } },