-
Notifications
You must be signed in to change notification settings - Fork 6
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
S138 bulk connection instructions #222
Conversation
default = {} | ||
|
||
# TODO: fix these validations; logically correct, but Terraform doesn't allow validation conditions | ||
# to depend on values of other variables |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI if you want to include the link of the issue to support this on their GH -> hashicorp/terraform#25609
per_setting_instructions_text = length(var.settings_to_provide) > 0 ? "\n${join("\n", tolist(local.per_setting_instructions))}" : "" | ||
} | ||
|
||
resource "local_file" "todo-worklytics-connection" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is already present in worklytics-psoxy-connection
module; not sure if intended or it is duplicated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, it's a more general case. the other one has a url
param that's not relevant for the bulk case.
Features
Change implications