Closed jtimpe closed 1 month ago
Attention: Patch coverage is 93.93939%
with 2 lines
in your changes are missing coverage. Please review.
Project coverage is 92.39%. Comparing base (
2c52e80
) to head (5a163d9
). Report is 7 commits behind head on develop.
Emails are coming through and looking good 🚀
@reitermb could you remind me -- is there an adjacent UX ticket that updates the e-mail notifications page related to this change? I'm also curious if we have prepared language about when these will and wont get delivered? (e.g. not delivered if files are still in pending status)
Summary of Changes
Pull request closes #2509. Updates the
send_data_submitted_email
toHow to Test
SENDGRID_API_KEY
in your.env
to a valid api key (i can provide one if you reach out). ensureSENDGRID_SANDBOX_MODE_IN_DEBUG
remainsFalse
.Deliverables
More details on how deliverables herein are assessed included here.
Deliverable 1: Accepted Features
Checklist of ACs:
lfrohlich
and/oradpennington
confirmed that ACs are met.Deliverable 2: Tested Code
CodeCov Report
comment in PR)CodeCov Report
comment in PR)Deliverable 3: Properly Styled Code
Deliverable 4: Accessible
iamjolly
andttran-hub
using Accessibility Insights reveal any errors introduced in this PR?Deliverable 5: Deployed
Deliverable 6: Documented
Deliverable 7: Secure
Deliverable 8: User Research
Research product(s) clearly articulate(s):