SSF-188 Automated Order Lifecycle Emails

[Juwang110]

ℹ️ Issue

Closes https://vidushimisra.atlassian.net/jira/software/projects/SSF/boards/1?jql=assignee%20%3D%20712020%3A10ef9ad9-e290-4bbd-8c4b-cb215e8d449a&selectedIssue=SSF-188

📝 Description

Email templates for:

• Pantry’s Food Request has a matched order
• FM’s donation has been matched to an order
• Food Request closed

✔️ Verification

BEFORE TESTING: Add 2 new environment variables:

AWS_SES_SENDER_EMAIL (set this to one of your emails, this will be the address that sends the emails for you to verify)
SEND_AUTOMATED_EMAILS (switch this to true to turn on Cognito account creation and email sending permissions)
Add your email that you put in the AWS_SES_SENDER_EMAIL into the following AWS SES Identities: https://us-east-2.console.aws.amazon.com/ses/home?region=us-east-2#/identities

Tested each workflow to ensure the proper sender, subject, message, attachments were all there.

Same testing as #127

I verified it worked via new tests and testing email functionality with postman.

🏕️ (Optional) Future Work / Notes

For food request closed the functionality is as follows (per Yurika):

For manual close (closeRequest endpoint):

• coordinator = volunteer who hit endpoing with req.user.id

For auto close (updateRequestStatus):

• coordinator = the last volunteer who touched an associated order of that request that was most recently delivered.

[sam-schu]

For sending the emails:

1. For consistency with our other emails, can we send the emails outside of the transaction, so the order is still created even if an email fails to send? (We do plan to add better handling for failed emails across the board)
2. Can we try sending both emails, even if one fails?

[sam-schu]

Confirmed the emails are properly sent! A few comments on the contents:

• Can we link the text "log into the platform" rather than including the link at the end of the email?
• Can we turn the coordinator email into a link?
• For the manufacturer email, can we make sure the different address lines appear on different lines in the email body, and add a comma between city and state?

814 Cedar Hollow Way
[Unit 1]
Madison, WI 53711
[US]

[dburkhart07]

sorry this is so many comments lol. should hopefully be good after this <3

[dburkhart07]

Couple last things.

IMPORTANT: Wait for Yurika's clarification before making some changes to updateRequestStatus

[sam-schu]

See comments, thanks!

[dburkhart07]

lgtm!! thanks for working so hard on this one!!!