search

badarsebard / terraform-provider-xsoar

Apache License 2.0
9 stars 3 forks source link

Propagation labels are not optional #5

Closed fancybear-dev closed 2 years ago

fancybear-dev commented 2 years ago

Documenation says propagation labels are optional, but because of a bug - this is currently not the case. I created this ticket so if anyone else encounters this, they know we have this issue as well.

We currently just pass the "all" label by default, as we don't need the propagation labels in our integrations.

image

jrauen commented 2 years ago

without the Computed: true option on the propagation_labels attribute the provider was unable to set the value received from the API in the state, creating the inconsistent result. please pull the latest commit and see if this resolves your issue.

fancybear-dev commented 2 years ago

Have confirmed just now, this fixes the bug we were having! Thanks for the quick response!

Thank you for the explanation for your fix as well, this helps me understand the issue and fix it myself in the future 😊. All this provider development in Go is still quite new for me...