[SSF-180, 189, 193] - Automated Emails Implementation#154
Open
swarkewalia wants to merge 14 commits intomainfrom
Open
[SSF-180, 189, 193] - Automated Emails Implementation#154swarkewalia wants to merge 14 commits intomainfrom
swarkewalia wants to merge 14 commits intomainfrom
Conversation
Juwang110
reviewed
Apr 18, 2026
sam-schu
requested changes
Apr 20, 2026
dburkhart07
changed the title
Juwang110
requested changes
May 6, 2026
| message.bodyHTML, | ||
| ); | ||
| } catch { | ||
| // email failed — still count as a recurrence and move on |
There was a problem hiding this comment.
In these cases should we be logging something? @sam-schu thoughts?
| schedule indicates a new donation submission is due. | ||
| </p> | ||
| <p> | ||
| When you have a moment, please log into your account and submit your current |
There was a problem hiding this comment.
For all these email templates can you add a link on "log into your account" for the login page? Sam asked me to do so on my PR #162
| message.subject, | ||
| message.bodyHTML, | ||
| ); | ||
| } catch (e) { |
| message.subject, | ||
| message.bodyHTML, | ||
| ); | ||
| } catch (e) { |
| setSelectedDonationId(initialDonationId); | ||
| fetchItemsForDonation(initialDonationId); | ||
| } | ||
| }, [isOpen, initialDonationId]); |
| Dialog, | ||
| Flex, | ||
| Portal, | ||
| Spinner, |
| foodRescue: item.foodRescue, | ||
| })), | ||
| }; | ||
| console.log(dto); |
| <ChevronDown size={20} /> | ||
| </Flex> | ||
|
|
||
| {isDropdownOpen && ( |
There was a problem hiding this comment.
when i open the dropdown it covers the submit button and overall the modal is very short, can we make it taller to accomodate both like in the design
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ℹ️ Issue
Closes #180, #189, and #193
📝 Description
Implemented email templates for:
Also updated the recurring donation reminder to use this template on the cron job instead of logging. The template uses this new query parameter,
resubmitDonationIdwhich automatically opens up the modal for that id, with all the donation items from it filled in (if there is a match).✔️ 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
Frontend page:
🏕️ (Optional) Future Work / Notes
emails go to spam :(