-
Notifications
You must be signed in to change notification settings - Fork 668
DYN-8203: Add a button to copy compatibility info from previous version #16831
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-8203
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
...amoCoreWpf/Views/PackageManager/Components/PackageManagerWizard/PackageManagerWizard.xaml.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.
...amoCoreWpf/Views/PackageManager/Components/PackageManagerWizard/PackageManagerWizard.xaml.cs
Outdated
Show resolved
Hide resolved
...amoCoreWpf/Views/PackageManager/Components/PackageManagerWizard/PackageManagerWizard.xaml.cs
Show resolved
Hide resolved
...amoCoreWpf/Views/PackageManager/Components/PackageManagerWizard/PackageManagerWizard.xaml.cs
Outdated
Show resolved
Hide resolved
|
Looks good with one comment. |
Comment addressed now. |
reddyashish
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ivaylo-matov. LGTM.

Purpose
This PR addresses DYN-8203
Currently,
PackageManagerWizardonly receives the current/cached version’s compatibility matrix viaPublishPackageViewModel.CompatibilityMatrixand it is not provided with compatibility matrices for prior versions. This change updates the Dynamo side to include the package’s full versions list (including compatibility_matrix per version) in the same update message sent to the wizard and the website UI. This allows the “Copy from” workflow to use compatibility info from any published version.This PR should be reviewed alongside the related PRs in the
PackageManagerWizardandPackageManagerWebsiteUIrepos:Declarations
Check these if you believe they are true
Release Notes
Updated logic to push all package versions to the Package Manager so users can copy compatibility matrices from previous versions.
Reviewers
@zeusongit
@DynamoDS/eidos
FYIs
@emru-y
@dnenov