-
Notifications
You must be signed in to change notification settings - Fork 63
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.
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
Allow AWS SES sourceArn to be set #4177
Conversation
part of #4173 Allows mail to be sent via AWS SES with a authorised identity
Has a paired Helm PR FlowFuse/helm#438 Need to work out a way to test this before merging, but should be able to test with localfs and the right AWS credentials |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4177 +/- ##
==========================================
- Coverage 78.62% 78.61% -0.02%
==========================================
Files 286 286
Lines 13097 13100 +3
Branches 2926 2928 +2
==========================================
Hits 10298 10298
- Misses 2799 2802 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Paired with FlowFuse/helm#438 for the helm config |
Both this and helm prs approved. Will leave merging at your discretion @hardillb |
part of #4173
Description
Allows mail to be sent via AWS SES with a authorised identity
Related Issue(s)
#4173
Checklist
flowforge.yml
?FlowFuse/helm
to update ConfigMap TemplateFlowFuse/CloudProject
to update values for Staging/ProductionLabels
area:migration
label