-
Notifications
You must be signed in to change notification settings - Fork 634
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
MD File Localization Proposal #15543
base: master_cl
Are you sure you want to change the base?
MD File Localization Proposal #15543
Conversation
During my research about adding the .resx files to all the projects for each language noticed that also we need to have the fallback_docs folder for each language. Then in this PR I'm proposing a change in the DynamoCore.csproj file for copying the md files to the fallback_docs inside each language folder, not sure if this change is needed due that we need to be sure that localization team has the changes for translating the md files once they are in this new location.
@ZbynekHanak-RWS any comment on this? |
Hi @RobertGlobant20 , thank you for this proposal. I created a mapping list of Dynamo Core resources - Dynamo CL - Mapping cBuild source to CL source . Seems few more sources need to be globalized, mainly we need to update folder structure. It can be filtered by "Source path/folder structure needs to be updated/globalized?" column. One of these issues is related to .md files installed to fallback_docs sub-folder for each language coming from doc\distrib\NodeHelpFiles folder. Would it be possible to move the source English .md files to en-US folder (ex. doc\distrib\NodeHelpFiles\en-US)? I will update CL then to deliver the localized .md files to language folders (ex. …\NodeHelpFiles\cs-CZ). Thank you |
Yes, we can move the md files to the folder |
@QilongTang when are we going to stop messing around with all these files and start using resource binaries? |
Thanks @mjkkirschner for double checking, yeah the plan for Morpheus was updating the localization process first to be source level localization, then look at if we can use resource files in 3.5 or 3.6 release. |
moving md files to the en-US folder and updating the DynamoCore.csproj for copying the files from the new location.
I've submitted a change for moving the files from |
@ZbynekHanak-RWS based in this request, is this the expected change? |
Hi @RobertGlobant20 , the change looks good. When the .md files will be in en-US folder I can update the .adsk/ContinuosLocalization.yaml config to allow CL to process these. Looks there will be need to update the DynamoCore.csproj project to pickup the localized .md files later. Seems now it copies the en-US .md to all localized folders using NodeHelpMDFiles property. Thanks |
Fix that will be copying all the md files inside each language folder like Dynamo\doc\distrib\NodeHelpFiles\en-US to the output path: Dynamo\bin\AnyCPU\Debug\en-US\fallback_docs
@ZbynekHanak-RWS I've submitted a fix for copying all the md files inside each language folder, For example copying from |
@ZbynekHanak-RWS any comment in the latest changes? |
Hi @RobertGlobant20 , it looks good for all the .md files that will be copied from the localized folders in repository to the localized fallback_docs output paths. Please note the .sat and .txt files are not localized. Can you just copy these from source en-US to localized output path instead? For example copying the .sat and .txt from |
I've updated the csproj file for using only one entry for copying the SAT and TXT files from en-US folder to each folder language.
@ZbynekHanak-RWS I've submitted a new commit for using just one source (en-US folder) when copying the SAT and TXT files, please let me know your thoughts about this change. |
Hi @RobertGlobant20 , yes, this change for the SAT and TXT files looks good. I think we can move forward to setup the MD files localization once this PR is merged to master_cl. Thank you |
@ZbynekHanak-RWS could you give me more details of what will be the next steps for the master_cl branch? Due that the master_cl branch was created by localization team, I have the next questions:
Thanks |
Hi @RobertGlobant20 , This master_cl was created to setup CL (Continuous Localization) pipeline for Dynamo Core product.
Meanwhile, Localization team will support the existing localization process to avoid any issue with the master branch. Thank you. |
Purpose
Proposal for changes in localizing md files in Dynamo
During my research about adding the .resx files to all the projects for each language noticed that also we need to have the fallback_docs sub-folder for each language.
Then in this PR I'm proposing a change in the DynamoCore.csproj file for copying the md files to the fallback_docs inside each language folder, not sure if this change is needed due that we need to be sure that localization team has the changes for translating the md files once they are in this new location.
Declarations
Check these if you believe they are true
*.resx
filesRelease Notes
Proposal for changes in localizing md files in Dynamo
Reviewers
@QilongTang
@ZbynekHanak-RWS
FYIs