You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
@reddyashish@Amoursol I don't think we should include the name package in this extension - right now it only supports packages of course - here is my reasoning:
The intention is for this extension/feature to handle more dependency types than packages - like zero touch, custom nodes, hosts themselves (like core Revit library), etc.
The name now includes PackageManager .... that seems like a confusing overload of the same name as the Dynamo package manager, as most users simply refer to it as package manager
what do you guys think of Workspace Requirements? or Requirements Manager or Required Libraries?
@reddyashish have you also looked through the tooltips and messages?
@mjkkirschner Yes, I agree with you. Sol has suggested the name "Workspace Manager" but I was thinking we can make it more clear. Required Libraries looks like a good choice but I was hoping the work "Workspace" to be included as these are specifically for each workspace.
Also regarding the tooltips, there is only one tooltip message for the refresh icon which says "Click to recalculate workspace dependencies". I will change this accordingly once we finalize the name for the extension
@reddyashish@mjkkirschner@Amoursol@kronz How about simply Workspace References, Reference is a widely used term already and Visual Studio use it as well for listing what current coding project is depending on in order to build and work. I would like to also limit the name as short as possible and as catchy as possible for future workshops and answer bar duty. My 2 cents.
Also, I vote against required libraries because it is not always 100% accurate, for example, if the graph is referencing a custom node from a package which is built on a bunch of default nodes, the user could still remove such required reference and build it by themselves. So I believe technically it is not really required but referenced to make the process simpler.
@QilongTang I like Workspace References as well because it can include any reference type if we want to extend it further in the future and the name still makes sense. (ie file references, cloud references, single nodes, libraries etc)
FWIW, you might look at what similar functionality is called in our host applications. For instance, Revit has "Manage Links", Autocad has a "Data Link Manager", although "Manager" is problematic with the name collision with the PM. Could also check with HIG folks if there is a standardizing language (Tom Vollaro would be a good person to start with"
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
This PR is to make simplify the terminology used in the workspace dependency extension.
Task: https://jira.autodesk.com/browse/DYN-2240
Looking for suggestions.
Declarations
Check these if you believe they are true
*.resx
filesReviewers
@DynamoDS/dynamo