DYN-: Dyn 9702 templates to home screen#17107
Draft
Chloepeg wants to merge 39 commits into
Draft
Conversation
- initial changes to the Dynamo backend to consume and show Dynamo Template files on HomePage - created StartPage container for Template files - added backend infrastructure to send Template files data to frontend on load
Add NewWorkspaceWithTemplate method to HomePage that creates a new workspace and inserts a template file, enabling the sidebar to open templates in a new editable workspace.
…m/Chloepeg/Dynamo into DYN-9702-templates-to-home-screen
- initial changes to the Dynamo backend to consume and show Dynamo Template files on HomePage - created StartPage container for Template files - added backend infrastructure to send Template files data to frontend on load
Add NewWorkspaceWithTemplate method to HomePage that creates a new workspace and inserts a template file, enabling the sidebar to open templates in a new editable workspace.
Add root folder bundle file to embedded resources
…m/Chloepeg/Dynamo into DYN-9702-templates-to-home-screen
Adds the missing TemplateFiles property to StartPageViewModel
Fixes build error
…workspace" This reverts commit e8ecae4.
remove merge conflict marker
…mpiles Fix DynamoCoreWpf.csproj DynamoHome embed and duplicate compiles
StartPage - Pulled “build one StartPageListItem from a path” into AddTemplateListItemFromPath so LoadTemplates() stays simpler - Made the templateFiles ObservableCollection readonly (only assigned in the constructor). - Renamed the local List<string> to rootDynPaths so it doesn’t shadow the templateFiles field. - Removed “&& this != null” on the non-empty check (always true in an instance method). Homepage - Replaced async void with async Task on LoadGraphs, LoadTemplates, SendSamplesData, SendTemplateData, and SendRecentGraphsData (Sonar S3168). - LoadingDone and RecentFiles_CollectionChanged use _ = Method() where the caller must stay void; SendTemplateData awaits LoadTemplates, SendRecentGraphsData awaits LoadGraphs. - Renamed the local in SendTemplateData to items for clarity.
Remove unused dirPaths in LoadTemplates() (StartPage.xaml.cs).
Use StartsWith('.') instead of StartsWith(".") for extension checks
Replace TODO comment in SendRecentGraphsData() with a neutral comment
adds a TemplateFiles item and a Copy to $(OutputPath)templates\%(RecursiveDir), so local and CI builds lay out templates the same way as samples.
- initial changes to the Dynamo backend to consume and show Dynamo Template files on HomePage - created StartPage container for Template files - added backend infrastructure to send Template files data to frontend on load
Add NewWorkspaceWithTemplate method to HomePage that creates a new workspace and inserts a template file, enabling the sidebar to open templates in a new editable workspace.
Add root folder bundle file to embedded resources
- initial changes to the Dynamo backend to consume and show Dynamo Template files on HomePage - created StartPage container for Template files - added backend infrastructure to send Template files data to frontend on load
Adds the missing TemplateFiles property to StartPageViewModel
Fixes build error
…workspace" This reverts commit e8ecae4.
remove merge conflict marker
…mpiles Fix DynamoCoreWpf.csproj DynamoHome embed and duplicate compiles
StartPage - Pulled “build one StartPageListItem from a path” into AddTemplateListItemFromPath so LoadTemplates() stays simpler - Made the templateFiles ObservableCollection readonly (only assigned in the constructor). - Renamed the local List<string> to rootDynPaths so it doesn’t shadow the templateFiles field. - Removed “&& this != null” on the non-empty check (always true in an instance method). Homepage - Replaced async void with async Task on LoadGraphs, LoadTemplates, SendSamplesData, SendTemplateData, and SendRecentGraphsData (Sonar S3168). - LoadingDone and RecentFiles_CollectionChanged use _ = Method() where the caller must stay void; SendTemplateData awaits LoadTemplates, SendRecentGraphsData awaits LoadGraphs. - Renamed the local in SendTemplateData to items for clarity.
Remove unused dirPaths in LoadTemplates() (StartPage.xaml.cs).
Use StartsWith('.') instead of StartsWith(".") for extension checks
Replace TODO comment in SendRecentGraphsData() with a neutral comment
adds a TemplateFiles item and a Copy to $(OutputPath)templates\%(RecursiveDir), so local and CI builds lay out templates the same way as samples.
…m/Chloepeg/Dynamo into DYN-9702-templates-to-home-screen
When selecting listed template files, open them as new graphs from their contents instead of treating the saved template path as a normal open. StartPage and HomePage now package the open argument as a tuple (path, false, true) for templates and pass that to DynamoViewModel.OpenCommand. DynamoViewModel.CanOpen was updated to extract the file path from either string, Tuple<string,bool>, or Tuple<string,bool,bool> so command validation works with the new packed arguments. A small helper IsListedHomePageTemplate was added to HomePage to detect listed templates.
Bump the embedded Dynamo Home package to 1.0.30 so Dynamo consumes the Home UI with template support
github-actions
Bot
changed the title
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-9702
3 tasks
|
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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 addresses DYN-9702 https://jira.autodesk.com/browse/DYN-9702 The changes in the code aim to add host-side support for Dynamo Home template files, enabling Dynamo to supply template data to Dynamo Home and ensure selected templates open as new editable workspaces.
This PR works with : DynamoDS/DynamoHome#89
Changes :
Added support for passing template metadata from Dynamo to Dynamo Home.
Declarations
Check these if you believe they are true
Release Notes
Added Dynamo-side support for template files in Dynamo Home. Dynamo now copies shipped templates into the build output, passes template metadata to the Home UI, and opens selected templates as new editable workspaces instead of opening the template file directly.
Updated the Dynamo/Home integration to consume @dynamods/dynamo-home 1.0.30, so the production embedded Home page uses the template-enabled UI rather than relying only on local development mode.
Reviewers
@zeusongit
@DynamoDS/eidos
FYIs
@dnenov
@johnpierson
@jnealb