New resource: azurerm_api_management_property

[mbfrahry]

This PR adds azurerm_api_management_property to the provider.

=== RUN   TestAccAzureRMAPIManagementProperty_basic
=== PAUSE TestAccAzureRMAPIManagementProperty_basic
=== RUN   TestAccAzureRMAPIManagementProperty_update
=== PAUSE TestAccAzureRMAPIManagementProperty_update
=== CONT  TestAccAzureRMAPIManagementProperty_basic
=== CONT  TestAccAzureRMAPIManagementProperty_update
--- PASS: TestAccAzureRMAPIManagementProperty_basic (1603.38s)
--- PASS: TestAccAzureRMAPIManagementProperty_update (1992.85s)

[katbyte]

Thanks for the PR @mbfrahry,

Commented on a couple minor mistakes I think need fixing, but other then that LGTM

[tombuildsstuff]

One minor comment in addition to @katbyte's but this otherwise LGTM 👍

[tombuildsstuff]

I feel like it could be worth a yellow info box here explaining this isn't making it sensitive within Terraform e.g.

~> **NOTE:** setting the field `secret` to `true` doesn't make this field sensitive in Terraform, instead it ...

[mbfrahry]

Done!

[ghost]

This has been released in version 1.23.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example:

provider "azurerm" {
	version = "~> 1.23.0"
}
# ... other configuration ...

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!