fix: Allow setup eventbridge without notifications (#160)

breitsmiley · web-flow · commit 31b8e9dcd179 · 2022-05-18T16:27:57.000+02:00
diff --git a/modules/notification/main.tf b/modules/notification/main.tf
@@ -7,7 +7,7 @@ locals {
 }
 
 resource "aws_s3_bucket_notification" "this" {
-  count = var.create && (length(var.lambda_notifications) > 0 || length(var.sqs_notifications) > 0 || length(var.sns_notifications) > 0) ? 1 : 0
+  count = var.create ? 1 : 0
 
   bucket = var.bucket
 

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ locals {`
`7`	`7`	`}`
`8`	`8`
`9`	`9`	`resource "aws_s3_bucket_notification" "this" {`
`10`		`- count = var.create && (length(var.lambda_notifications) > 0 \|\| length(var.sqs_notifications) > 0 \|\| length(var.sns_notifications) > 0) ? 1 : 0`
	`10`	`+ count = var.create ? 1 : 0`
`11`	`11`
`12`	`12`	`bucket = var.bucket`
`13`	`13`