Sarah French


@BBBmau As follow up to my last review, I've got some questions: 1. Are we going to use a feature branch for this work or not? Either choice is fine...

I'll leave review to @roaks3. If this PR addresses ordering diffs on this field by converting it to a Set in 6.0.0 then this PR will also need to remove...

>@SarahFrench this is a plugin-framework resource - any chance you happen to know offhand what might be going on here in terms of the error not being surfaced? I just...

I started playing with a debugger and found that an error is swallowed - it looks like it's being handled in the snippet above but it's not actually returned to...

After testing the provider with the fix in the PR above we found that the data source returned an error like: ``` ╷ │ Error: Error setting access_token │ │...

Hi @bitgandtter - the fix for this issue is related to a wider fix for how the provider is configured. There's a draft PR open for that fix here (https://github.com/GoogleCloudPlatform/magic-modules/pull/11903)...

~TODO: Tests should make assertions about the schema of the store itself being in the returned schema.~ See [d19eb83](https://github.com/hashicorp/terraform/pull/37959/commits/d19eb839cc03537c8795ea0243ebfc932da01527)

Note from triage: The [API docs currently only show SAS token](https://cloud.google.com/storage-transfer/docs/reference/rest/v1/TransferSpec#AzureCredentials) as a method of authentication with Azure, but [this PR](https://github.com/GoogleCloudPlatform/magic-modules/pull/11893) looks like it's implementing federated identity config as described...

+1 to what @mildwonkey said. If a public repo is not an option, here is an example of enabling public repos to access tools from a private repo from elsewhere...