Add APIVersion and APIVersionSetID to import

[sam-cogan]

Adds the APIVersion and APIVersionSetID to the REST call for creating an API using an import from a file (swagger, WSDL etc.). This fixes the bug in #4985 that would result in creation failing if attempting to create more than 1 version of a single API with the same path.

Fixes #4985

[tombuildsstuff]

hey @sam-cogan

Thanks for this PR - taking a look through this is looking pretty good, if we can fix up the comments (mostly adding back in the missing links) then this otherwise LGTM 👍

Thanks!

[sam-cogan]

@tombuildsstuff I've updated the code to move the validation earlier, but not sure what you meant by "adding back missing links"

[sam-cogan]

@tombuildsstuff is there anything left that needs doing to get this merged?

[tombuildsstuff]

chatting offline: this is fine, I'd confused the comment when writing the PR summary 👍

[tombuildsstuff]

Tests pass:

[tombuildsstuff]

LGTM - thanks for this @sam-cogan

[ghost]

This has been released in version 1.39.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.39.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 👉 [email protected]. Thanks!