DYN-7570 Purple package nodes overlay for non-custom nodes#15532
DYN-7570 Purple package nodes overlay for non-custom nodes#15532QilongTang merged 3 commits intoDynamoDS:masterfrom
Conversation
- now adds package name check to determine if a node belongs to a package
- for whatever reason, only custom functions were allowed to be given PackageName - removed this limitation
UI Smoke TestsTest: success. 11 passed, 0 failed. |
I think you mean When reviewing a different task, @aparajit-pratap has indicated that DynamoForma does not follow such structure and use a dedicated dll/helper to load all the nodes. Maybe that's the reason this is still going to break for Forma. |
There was a problem hiding this comment.
See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-7570
Yes, exactly, apologies for the poor wording there. I hope this means that for the majority of the cases, the above should be sufficient. |
Purpose
Continuation of and older task that might not have been fully addressed the first time around - a link to the original PR can be found here: #14507
Context
Currently, only
CustomFunctionswere flagged up as 'Package' nodes in the UI. This PR attempts to add functions coming from Packages, regardless of their node type.Changes (the gif is a bit long, sorry):
Limitations
In the example below, a node coming from the
Formapackage AssemblyName is not found in the list of packages insdie theNodePackageDictionary:Conclusion
For this function to work correctly, package authors must make sure that the nodes are contained within the same assembly that the package will be listed under by the
PackageExtension(please, correct me if I am wrong on this one).Declarations
Check these if you believe they are true
*.resxfilesRelease Notes
PackageNamecheck to mark a node as aCustomNodeIsCustomFuctionset to trueReviewers
@QilongTang
@zeusongit
@reddyashish
FYIs
@Amoursol