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

Merge branch 'restrict-email-results-to-10-only' into 'master'

Browse Source 
limit number of real estate results in email to 10

See merge request saburly/marketalarm/web!28
This commit was merged in pull request #28.
This commit is contained in:
Bilal Catic
2019-09-10 15:25:29 +00:00
parent de3ea71342 1f79e1d555
commit 0b78254573
1 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/helpers/awsEmail.js
View File

@@ -143,12 +143,15 @@ const sendBulkEmail = async marketAlerts => {
}; };
} }
if (groupedRERequests[marketAlert.request].marketAlertArray.length < 10) {
groupedRERequests[marketAlert.request].marketAlertArray.push({ groupedRERequests[marketAlert.request].marketAlertArray.push({
url: marketAlert.url, url: marketAlert.url,
title: marketAlert.title, title: marketAlert.title,
id: marketAlert.id id: marketAlert.id
}); });
}
}); });
for (request in groupedRERequests) { for (request in groupedRERequests) {
const marketAlert = groupedRERequests[request]; const marketAlert = groupedRERequests[request];
let extractedData = toAWSArray(marketAlert.marketAlertArray); let extractedData = toAWSArray(marketAlert.marketAlertArray);