-
Notifications
You must be signed in to change notification settings - Fork 24
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
Cannot manage launchdarkly_feature_flag_environment with rules containing boolean clauses #51
Comments
Hi @mvisonneau, This is a good find. I've filed an internal ticket to get this fixed. I'll be sure to update you here when the updated release is available. Thanks, |
Thanks, I assume it has something to do with: https://github.com/launchdarkly/terraform-provider-launchdarkly/blob/master/launchdarkly/clause_helper.go#L26:L26 |
Yeah, you got it. In order to avoid introducing a breaking change we will likely add a Unfortunately this means that we will not be able to support polymorphic types in the values array in a v1.X release, but I think this is acceptable for the majority of use cases. For v2 we will likely break the schema to support polymorphic value types. |
BUG FIXES: Fixed a bug preventing number and boolean values in targeting rules clauses from working. The new value_type attribute must be set in order to utilize number and boolean values. All values for a given targeting rule clause must be of the same type. #51
Hi @mvisonneau, I just published v1.5.1 of the provider which includes a fix for this issue. After upgrading, you will have to set the rule clause's Thanks for reporting this, |
Brilliant, thanks a lot @ldhenry, it works like a charm now 🙇 👌 |
Assuming the following existing configuration:
Importing it into the statefile or attempting to perform plan upon this resource results into the following error:
The text was updated successfully, but these errors were encountered: