Don't show phone number as target for email notifs

Fixes https://github.com/vector-im/riot-web/issues/3518
pull/3530/head
David Baker 2017-03-28 16:24:54 +01:00
parent d696196d72
commit 83b7f94609
1 changed files with 5 additions and 8 deletions

View File

@ -719,20 +719,17 @@ module.exports = React.createClass({
); );
} }
var emailNotificationsRow; const emailThreepids = this.props.threepids.filter((tp) => tp.medium == "email");
if (this.props.threepids.filter(function(tp) { let emailNotificationsRow;
if (tp.medium == "email") { if (emailThreepids.length == 0) {
return true;
}
}).length == 0) {
emailNotificationsRow = <div> emailNotificationsRow = <div>
Add an email address above to configure email notifications Add an email address above to configure email notifications
</div>; </div>;
} else { } else {
// This only supports the first email address in your profile for now // This only supports the first email address in your profile for now
emailNotificationsRow = this.emailNotificationsRow( emailNotificationsRow = this.emailNotificationsRow(
this.props.threepids[0].address, emailThreepids[0].address,
"Enable email notifications ("+this.props.threepids[0].address+")" "Enable email notifications ("+emailThreepids[0].address+")"
); );
} }