diff --git a/server/api/harvestingrequests.js b/server/api/harvestingrequests.js index 592fdc28..4e3d6ae6 100644 --- a/server/api/harvestingrequests.js +++ b/server/api/harvestingrequests.js @@ -80,7 +80,7 @@ const addRequest = (req, res) => { }).then(() => { // ignore return value of Mailgun call // send notification to LibreTexts team - return mailAPI.sendOERIntRequestAdminNotif(finalizedRequest.name, finalizedRequest.title); + return mailAPI.sendOERIntRequestAdminNotif(finalizedRequest.name, finalizedRequest.email, finalizedRequest.title); }).then(() => { // ignore return value of Mailgun call return res.send({ diff --git a/server/api/mail.js b/server/api/mail.js index e1128b4e..582413d8 100644 --- a/server/api/mail.js +++ b/server/api/mail.js @@ -174,12 +174,13 @@ const sendOERIntRequestConfirmation = (requesterName, recipientAddress, resource * NOTE: Do NOT use this method directly from a Conductor API route. Use internally * only after proper verification via other internal methods. * @param {String} requesterName - the requesting user's name ('firstName' or 'firstName lastName') + * @param {String} requesterEmail - the requesting user's email * @param {String} resourceTitle - the resource's title/name * @returns {Promise} a Mailgun API Promise */ -const sendOERIntRequestAdminNotif = (requesterName, resourceTitle) => { - let textToSend = `Attention: ${requesterName} has submitted a new OER Integration Request for "${resourceTitle}". This request is available in Conductor.` + autoGenNoticeText; - let htmlToSend = `

Attention:

${requesterName} has submitted a new OER Integration Request for "${resourceTitle}".

This request is available in Conductor.

` + autoGenNoticeHTML; +const sendOERIntRequestAdminNotif = (requesterName, requesterEmail, resourceTitle) => { + let textToSend = `Attention: ${requesterName ? requesterName : requesterEmail} has submitted a new OER Integration Request for "${resourceTitle}". This request is available in Conductor.` + autoGenNoticeText; + let htmlToSend = `

Attention:

${requesterName ? `${requesterName} (${requesterEmail})` : requesterEmail} has submitted a new OER Integration Request for "${resourceTitle}".

This request is available in Conductor.

` + autoGenNoticeHTML; return mailgun.messages.create(process.env.MAILGUN_DOMAIN, { from: 'LibreTexts Conductor ', to: ['info@libretexts.org'],