Skip to content

update confirmation email test to use SES rather than notify - #324

Draft
alice-carr wants to merge 1 commit into
mainfrom
use-ses-for-confirmation-email
Draft

update confirmation email test to use SES rather than notify#324
alice-carr wants to merge 1 commit into
mainfrom
use-ses-for-confirmation-email

Conversation

@alice-carr

Copy link
Copy Markdown
Contributor

What problem does this pull request solve?

we are switching from using notify to send confirmation emails to using SES - this will keep it in line with the copy of answers feature.

This pr updates the end to end tests so that they now check s3 rather than notify for the confirmation email. This includes adding a new confirmation email that has already been set up in ses.

Trello card: https://trello.com/c/GfBTJbNL/3154-use-ses-to-send-all-confirmation-emails

Things to consider when reviewing

you can review this by running the e2e tests against the forms-runner branch: use-ses-to-send-all-confirmation-emails

  • Ensure that you consider the wider context.
  • Does it work when run on your machine?
  • Is it clear what the code is doing?
  • Do the commit messages explain why the changes were made?
  • Has all relevant documentation been updated?

we are switching from using notify to send confirmation emails to using SES - this will keep it in line with the copy of answers feature. This commit updates the end to end tests so that they now check s3 rather than notify for the confirmation email.
@alice-carr
alice-carr marked this pull request as draft August 4, 2026 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant