Fixed parsing of management group id

[simongh]

I think this addresses #8023

[ArcturusZhang]

Hi @simongh thanks for this PR!

This PR looks great to me. But could you please also add more test cases in this file: https://github.com/terraform-providers/terraform-provider-azurerm/blob/594d00c830cb375148a3b42753b65c42d3696191/azurerm/internal/services/managementgroup/parse/management_group_test.go?

I suppose add new cases with ManagementGroups segment should work.

Thanks!

[simongh]

Added a couple of tests with alternate casing

[ArcturusZhang]

LGTM 👍 thanks!

[dpbustos]

When will this PR be merged in?

[katbyte]

LGTM 👍

[ghost]

This has been released in version 2.28.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 = "~> 2.28.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!