Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Email Notifications [mimics main branch] #140

Draft
wants to merge 106 commits into
base: main
Choose a base branch
from
Draft

Email Notifications [mimics main branch] #140

wants to merge 106 commits into from

Conversation

coilysiren
Copy link
Contributor

@coilysiren coilysiren commented Oct 4, 2024 •
edited by github-actions bot
Loading

Ticket

See also: #141

Resolves navapbc/template-infra#690

Context

See #141 for a comprehensive rundown of the changes. This PR includes a hardcoded exception to make this PR branch mimic a main branch. That is, to make it deploy the Email Notifications resources.

Testing

image

Preview environment

rocketnova and others added 30 commits August 14, 2024 12:15
@rocketnova
Bring over WIP from template-infra branch
ab8e5a4
@rocketnova
Enable notifications
9ab6473
@rocketnova
Restrict access
e772c1e
@rocketnova
Conditionally return values based on verification method
18ee918
@rocketnova
Deal with pinpoint_email_channel role_arn
d62b062
@rocketnova
Separate a verified sender identity from a pinpoint app
391b603
@rocketnova
Refactor outputs and variables
9a607bb
@rocketnova
Add code for aws_sesv2_email_identity_policy
0c417ad
@rocketnova
Drop workspace prefix for notifications config
179b385
@rocketnova
Connect notifications and email-identity modules with app-config and …
79f1bb9 
…app/service
@rocketnova
Rework idp to remove dependency on data output which would otherwise …
c07cb33 
…require first running a -target run
@rocketnova
Add test email to satisfy tests
11eee11
@rocketnova
Fix IAM role name collision
a3e203b
@rocketnova
Grant Github Actions auth for notifications: ses & mobiletargeting
c649240
@rocketnova
Remove deprecated pinpoint identity role
5dbcda6
@rocketnova
Move email identity into resources subdir
f0a9759
@rocketnova
Refactor to support re-using existing email identities
7330a31
@rocketnova
Fix depends_on syntax
d7b5e35
@rocketnova
Simplify
3a80dbf
@rocketnova
Simplify and use domain name
f6a57ed
@rocketnova
Auto-add dns records for domain verification
06bae3a
@coilysiren
Merge branch 'main' into rocket/notifications
9a53c2c
@coilysiren
diff reduction
1e24692
@coilysiren
spacing and docs
64268b1
@coilysiren
use jsonencode
46dd4cd
@coilysiren
module path
0e4eb4c
@coilysiren
prefix
a3834ee
@coilysiren
try to fix verification records
2b7bda9
@coilysiren
try using count
d2c2bf3
@coilysiren
use route53 config from truss
33e654d
@coilysiren
narrow env var scope
31ff496
@coilysiren
domain_identity_arn
d421603
@coilysiren
cleanup some vars
52643a5
@coilysiren
validation + test
f2d43bf
@coilysiren
Revert "validation + test"
a6d6d83 
This reverts commit f2d43bf.
@coilysiren
use domain name as name
d8b89e2
@coilysiren
remove some name
8a10e95
@coilysiren
comment
7081ed2
@coilysiren
move stuff around
d1218bb
@coilysiren
ignore a bunch of stuff
98d8036
@coilysiren
fix domain vars
e3379d2
@coilysiren
testing
8a72123
@coilysiren
pull from correct var hopefully
592a188
@coilysiren
use default email config, instead of explicit one
09a192d
@coilysiren
various renames
9794f45
@coilysiren
revert idp stuff
1fa2229
@coilysiren
delete app first
5910294
@coilysiren
delete it please, thanks
31eb7b2
@coilysiren
update some var refs
fcfa6f4
@coilysiren
put everything back
b8cd1fe
@coilysiren
update vars
103f386
@coilysiren
reorg
fe2678d
@coilysiren
a var name
e1c6c2f
@coilysiren
Merge branch 'kai/notifications-2' into kai/notifications
e46c6b7
@coilysiren
typo
cd3088a
@coilysiren
bool
545c659
@coilysiren
testing env
b155ab2
@coilysiren
fix ifs
0584221
@coilysiren
fix domain input
e820044
@coilysiren coilysiren mentioned this pull request Nov 22, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lorenyu lorenyu lorenyu left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Email notifications
3 participants
@coilysiren @lorenyu @rocketnova