From 83b7f94609aee1e352dbf425787ac5d603c7b25e Mon Sep 17 00:00:00 2001 From: David Baker Date: Tue, 28 Mar 2017 16:24:54 +0100 Subject: [PATCH] Don't show phone number as target for email notifs Fixes https://github.com/vector-im/riot-web/issues/3518 --- src/components/views/settings/Notifications.js | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/src/components/views/settings/Notifications.js b/src/components/views/settings/Notifications.js index b5fc2f3d61..78bde820a1 100644 --- a/src/components/views/settings/Notifications.js +++ b/src/components/views/settings/Notifications.js @@ -719,20 +719,17 @@ module.exports = React.createClass({ ); } - var emailNotificationsRow; - if (this.props.threepids.filter(function(tp) { - if (tp.medium == "email") { - return true; - } - }).length == 0) { + const emailThreepids = this.props.threepids.filter((tp) => tp.medium == "email"); + let emailNotificationsRow; + if (emailThreepids.length == 0) { emailNotificationsRow =
Add an email address above to configure email notifications
; } else { // This only supports the first email address in your profile for now emailNotificationsRow = this.emailNotificationsRow( - this.props.threepids[0].address, - "Enable email notifications ("+this.props.threepids[0].address+")" + emailThreepids[0].address, + "Enable email notifications ("+emailThreepids[0].address+")" ); }