From 46ad1feb9680bd8c5075dba6f042459079809a7c Mon Sep 17 00:00:00 2001 From: Tom Moor Date: Tue, 25 Jun 2019 20:28:45 -0700 Subject: [PATCH] chore: Remove welcome email for Slack users (against TOS) --- server/models/User.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/server/models/User.js b/server/models/User.js index 22d18b56..d3ec078c 100644 --- a/server/models/User.js +++ b/server/models/User.js @@ -160,7 +160,13 @@ User.beforeSave(uploadAvatar); User.beforeCreate(setRandomJwtSecret); User.afterCreate(async user => { const team = await Team.findByPk(user.teamId); - sendEmail('welcome', user.email, { teamUrl: team.url }); + + // From Slack support: + // If you wish to contact users at an email address obtained through Slack, + // you need them to opt-in through a clear and separate process. + if (team.service !== 'slack') { + sendEmail('welcome', user.email, { teamUrl: team.url }); + } }); // By default when a user signs up we subscribe them to email notifications