Fix regions update #322
Merged
Fix regions update #322
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
DESCRIPTION: receiving 400 errors when trying to update or modify regions (answer groups) with the new version of the Terraform provider. Adding answer groups to a record currently works
The main issue is that TF can add empty elements in sets, so fixing that by ignoring elements where the name is empty.
Also the need to update tags and blocked_tags at the same time has been overcome, so removing that check.
Right now TestAccRecord_meta and TestAccRecord_meta_with_json are failing for me (on a PUT /v1/monitoring/jobs), but I get the same result with the master branch so it seems unrelated.