Skip to content

Conversation

AllyW
Copy link
Contributor

@AllyW AllyW commented Feb 8, 2025

The result of getExtensions from model needed to be applied

@AllyW AllyW changed the title add prop identifier add prop decorator parsing for x-ms-identifiers and x-ms-secret Feb 13, 2025
Comment on lines -795 to -797
if (item.type === "object" && !array.identifiers && getProperty(payloadModel.indexer.value as Model, "id") && getProperty(payloadModel.indexer.value as Model, "name")) {
array.identifiers = ["name"];
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should keep this logic for array of objects without identifiers

@AllyW AllyW merged commit 0f10cd1 into Azure:dev Feb 13, 2025
3 checks passed
@kairu-ms kairu-ms mentioned this pull request Feb 27, 2025
@AllyW AllyW deleted the add-prop-identifier branch February 28, 2025 06:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants