Skip to content

Conversation

@MicroFish91
Copy link
Contributor

@MicroFish91 MicroFish91 commented Jan 29, 2026

Closes #1206

Just use a UUID for the install tree items. We already pass the resource ID to the actual install command through a command arg anyway; we just need the ID to be unique in the tree view.

Copilot AI review requested due to automatic review settings January 29, 2026 19:47
@MicroFish91 MicroFish91 requested a review from a team as a code owner January 29, 2026 19:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes an error that occurred when expanding the "Source resources" node under a function app's identity node. The issue was caused by tree item ID conflicts when creating "install extension" action items for resources that appear in multiple locations in the tree.

Changes:

  • Replace resource ID-based tree item ID with UUID for install extension items in DefaultAzureResourceItem

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MicroFish91 MicroFish91 changed the title Use a UUID for the tree item id for installation tree items Use a UUID for extension install tree items Jan 29, 2026
@MicroFish91 MicroFish91 enabled auto-merge (squash) January 29, 2026 19:53
@MicroFish91 MicroFish91 merged commit d595dbe into main Jan 29, 2026
3 checks passed
@MicroFish91 MicroFish91 deleted the mwf/fix-1206-rg branch January 29, 2026 19:55
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.

An error occurs when expanding "Source resources" node

3 participants