Add to output identity block values

[zioproto]

Describe your changes

Add the values from the identity block in the module output
https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/cognitive_account

Issue number

#84

Checklist before requesting a review

• [ X] The pr title can be used to describe what this pr did in CHANGELOG.md file
• [ X] I have executed pre-commit on my machine
• [ X] I have passed pr-check on my machine

Thanks for your cooperation!

[zioproto]

@lonegunmanb Question: is it correct to create an output for each element of the identity block of the resource ? or I should have created a single output exposing the all block ?

https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/cognitive_account

[lonegunmanb]

@zioproto I think we need a try here since it's possible to have an account without identity block.

is it correct to create an output for each element of the identity block of the resource ? or I should have created a single output exposing the all block ?

Both works for me, I would say let's output the identity block:

output "cognitive_account_identity" {
  ...
  value = try(azurerm_cognitive_account.this.identity[0], null)
}

[lonegunmanb]

@zioproto I think we need a try here since it's possible to have an account without identity block.

is it correct to create an output for each element of the identity block of the resource ? or I should have created a single output exposing the all block ?

Both works for me, I would say let's output the identity block:

output "cognitive_account_identity" {
  ...
  value = try(azurerm_cognitive_account.this.identity[0], null)
}

[zioproto]

@lonegunmanb please take a look again and merge if this is now OK. Thank you

[lonegunmanb]

Thanks @zioproto , LGTM! 🚀