fix: the format for email notifications were incorrect

appvia · Apr 27, 2024 · 1e81cd3 · 1e81cd3
1 parent f579d87
commit 1e81cd3
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/locals.tf b/locals.tf
@@ -2,6 +2,13 @@
 locals {
   ## Indicates if slack notifications are enabled 
   enable_slack = var.notifications.slack != null ? true : false
+  ## Indicates if email notifications are enabled 
+  enable_email = var.notifications.email != null ? true : false
+
+  ## The configuration for email notifications if enabled 
+  email = local.enable_email ? {
+    addresses = var.notifications.email.addresses
+  } : null
 
   ## The configuration for slack notifications if enabled 
   slack = local.enable_slack ? {

diff --git a/main.tf b/main.tf
@@ -6,7 +6,7 @@ module "notifications" {
 
   allowed_aws_services = ["budgets.amazonaws.com", "lambda.amazonaws.com"]
   create_sns_topic     = var.create_sns_topic
-  email                = try(var.notifications.email.addresses, [])
+  email                = local.email
   slack                = local.slack
   sns_topic_name       = var.sns_topic_name
   tags                 = var.tags