Skip to content

Conversation

btrieger
Copy link
Contributor

Connectors were only properly storing url in the config in the tfstate file as they were being mapped to Tines.

Fix #388

@btrieger btrieger requested a review from dimuon September 26, 2023 21:27
dimuon
dimuon previously approved these changes Sep 27, 2023
Copy link
Contributor

@dimuon dimuon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contributing!
LGTM 👍 besides minor formatting issue.

Co-authored-by: Dmitry Onishchenko <8962171+dimuon@users.noreply.github.com>
Co-authored-by: Dmitry Onishchenko <8962171+dimuon@users.noreply.github.com>
@btrieger
Copy link
Contributor Author

Pushed an update with the formatting. Looks like you need to approve again. Odd that it looked fine in my ide

Copy link
Contributor

@dimuon dimuon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dimuon
Copy link
Contributor

dimuon commented Sep 27, 2023

Odd that it looked fine in my ide

It could be that your IDE uses spaces instead of tabs.

@btrieger btrieger merged commit 2a72f16 into elastic:main Sep 27, 2023
tobio added a commit to wandergeek/terraform-provider-elasticstack that referenced this pull request Oct 1, 2023
…ramework

* origin/main:
  Fix mapping of webhook connectors that is stored in tfstate (elastic#433)
  Bump version for v0.8.0
  Fix Kibana SLO panic `histogram_custom_indicator` (elastic#430)
  Bump github.com/hashicorp/terraform-plugin-sdk/v2 from 2.26.1 to 2.29.0 (elastic#422)
  Bump github.com/hashicorp/terraform-plugin-docs in /tools (elastic#384)
  Bump github.com/golangci/golangci-lint from 1.54.1 to 1.54.2 in /tools (elastic#409)
  Bump actions/checkout from 3 to 4 (elastic#421)
  fix: update component documentation (elastic#420)
  Add support for the .slack_api connector (elastic#419)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] elasticstack_kibana_action_connector webhook config not stored correctly in state
2 participants