Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incorrect display of map types #64

Open
theunrepentantgeek opened this issue Jun 17, 2023 · 0 comments · May be fixed by #66
Open

Incorrect display of map types #64

theunrepentantgeek opened this issue Jun 17, 2023 · 0 comments · May be fixed by #66

Comments

@theunrepentantgeek
Copy link
Contributor

In the generated documentation for Azure Service Operator (ASO), we noticed that the display names of fields with a map type weren't being simplified the same way that other types are.

For example,

On VirtualMachineIdentity_ARM we see a map type with a relative path for the type of map values:

userAssignedIdentities
map[string]./api/compute/v1api20220301.UserAssignedIdentityDetails_ARM

We expected this to display with just the name of the type, given it's in the current package:

userAssignedIdentities
map[string]UserAssignedIdentityDetails_ARM

Slice properties already use this simplified form - several examples can be seen on VirtualMachineInstanceView_STATUS_ARM:

disks
[]DiskInstanceView_STATUS_ARM

extensions
[]VirtualMachineExtensionInstanceView_STATUS_ARM

I've traced the cause of this to the typeDisplayName() function.

This was referenced Jun 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant