senaduka/old-web
1
0
Fork 0
Code Issues Pull Requests 5 Actions Packages Projects Releases Wiki Activity

Merge branch 'prevent-sending-emails-to-unsubscribed-users' into 'master'

Browse Source 
check if user is subscribed before sending email

See merge request saburly/marketalarm/web!69
This commit was merged in pull request #69.
This commit is contained in:
Bilal Catic
2019-11-05 06:54:59 +00:00
parent 8ea44f5fc7 310448dcb8
commit 91cda0ff0f
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/services/notificationService.js
View File

@@ -36,8 +36,8 @@ const notifyMatches = async (matches, dailyNotification = false) => {
const asyncSendEmailActions = [];
for (const id of searchRequestsToNotify) {
const { searchRequest, notifyNow } = matches[id];
if (notifyNow) {
const { email } = searchRequest;
const { email, subscribed } = searchRequest;
if (notifyNow && subscribed) {
const allMatchingRealEstates = matches[id].realEstates || [];
if (allMatchingRealEstates.length > 0) {
const emailContent = generateNotificationEmail(