-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
apimanagement: make description field required #14073
apimanagement: make description field required #14073
Conversation
Hello @Victorp99 - thank you for this PR, while the portal does seem to mark this property as required these tests are currently passing without a description being passed. Thus i'm not sure this property needs to be required? |
Hello @katbyte! I updated all of the tests that had the |
I think I misunderstood your question initially. I did not submit the issue this PR addresses, but this fix would allow for a similar user experience when provisioning this resource whether it be via the Portal or Terraform. |
It looks like the API endpoint used by the Azure SDK for Go utilized by Terraform does not require a value for the When looking through the documentation, I did find that the |
Thanks for looking into it in more detail @Victorp99 - will close this as requested |
Thank you for reviewing my PR! |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
Summary
This pull request makes the
Description
field required for theazurerm_api_management_product
resource.Fixes: #13856
Output from acceptance testing: