Move PackageManager UI to DynamoPackagesWPF view extension#16866
Move PackageManager UI to DynamoPackagesWPF view extension#16866
Conversation
…oject file Co-authored-by: QilongTang <3942418+QilongTang@users.noreply.github.com>
…y target Co-authored-by: QilongTang <3942418+QilongTang@users.noreply.github.com>
Co-authored-by: QilongTang <3942418+QilongTang@users.noreply.github.com>
Co-authored-by: QilongTang <3942418+QilongTang@users.noreply.github.com>
Co-authored-by: QilongTang <3942418+QilongTang@users.noreply.github.com>
Co-authored-by: QilongTang <3942418+QilongTang@users.noreply.github.com>
Co-authored-by: QilongTang <3942418+QilongTang@users.noreply.github.com>
…dependency Co-authored-by: QilongTang <3942418+QilongTang@users.noreply.github.com>
| /// </summary> | ||
| public class OpenPackageManagerEventArgs : EventArgs | ||
| { | ||
| private string tab; |
|
|
@copilot Rebase |
I've successfully rebased the branch onto the latest master (commit fee9456). The rebase resolved conflicts where master had modified some PackageManager files that were being moved/deleted in this PR. However, I'm unable to force-push the rebased commits due to permission restrictions. The git push returns a 403 error. Could you help push the rebased branch, or should I take a different approach? The local branch has been rebased and is ready - it just needs to be force-pushed to update the remote. |
|
@copilot Address merge conflicts |
Resolved conflicts: - DynamoView.xaml.cs: Kept deletion of PackageManager methods (moved to view extension) - PackageManagerTabControl.cs: Moved to correct location in DynamoPackagesWPF - Added PackageManagerTabControl.cs to DynamoPackagesWPF.csproj Co-authored-by: QilongTang <3942418+QilongTang@users.noreply.github.com>
Merge conflicts resolved in commit b05081f. Conflicts resolved:
The branch now includes all 103 commits from master while preserving the PackageManager UI migration. |
|




DynamoPackageManager UI was incorrectly housed in DynamoCoreWpf instead of its corresponding view extension project, violating the OOTB view extension pattern.
Changes
File Migration
DynamoCoreWpftoDynamoPackagesWPF:Architecture
Before:
After:
Key Decisions
PackageManagerSizeEventArgs,OpenPackageManagerEventArgs) moved toDynamoViewModelEvents.cs- they're part of the event contract, not UI implementationProject Files
DynamoCoreWpf.csproj(orphaned tags, misplaced closing elements)DynamoPackagesWPF.csprojwith all migrated files and npm build targetsResult
PackageManager UI now follows the same pattern as other OOTB view extensions (DocumentationBrowser, GraphNodeManager, etc.) with proper separation between core framework and extension UI.
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.