942 replace pinpoint with SES

[sean-navapbc]

Ticket

Resolves #{TICKET NUMBER OR URL}

Changes

What was added, updated, or removed in this PR.

Context for reviewers

Testing instructions, background context, more in-depth details of the implementation, and anything else you'd like to call out or ask reviewers.

Testing

Provide evidence that the code works as expected. Explain what was done for testing and the results of the test plan. Include screenshots, GIF demos, shell commands or output to help show the changes working as expected. ProTip: you can drag and drop or paste images into this textbox.

Preview environment for app-nextjs

• Service endpoint: https://p-231-app-nextjs-dev-1935800876.us-east-1.elb.amazonaws.com
• Deployed commit: e1700c3

Preview environment for app-flask

♻️ Environment destroyed ♻️

Preview environment for app

♻️ Environment destroyed ♻️

Preview environment for app-rails

♻️ Environment destroyed ♻️

[doshitan]

I don't think we need to pass the configuration set around do we? It should be set as the default for the sender already in infra/modules/notifications-email-domain/resources/main.tf.

We'll need to update various docs and comments (anywhere mentioning "Pinpoint"). But that could just be on the template PR.

[doshitan]

Looks to be working well via https://p-231-app-dev-1629703657.us-east-1.elb.amazonaws.com/email-notifications, which looks to be using the new config.

Can probably open up the template PR at this point.

[sean-navapbc]

I don't think we need to pass the configuration set around do we? It should be set as the default for the sender already in infra/modules/notifications-email-domain/resources/main.tf.

We'll need to update various docs and comments (anywhere mentioning "Pinpoint"). But that could just be on the template PR.

It's set as default on line 15 of infra/modules/notifications-email-domain/resources/main.tf