Skip to content
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

DYN-7480 Source-level Localization Pipeline - Node Help Docs #15747

Merged
merged 14 commits into from
Jan 9, 2025

Conversation

QilongTang
Copy link
Contributor

@QilongTang QilongTang commented Jan 8, 2025

Purpose

Enable node help docs source level localization pipeline. This PR includes:

  • yml file under .adsk folder specifying automation from localization team
  • reorganization of node help docs md files
  • initial batch of localized md files in each language folder (contributing the large number of files)

After this PR merged, changes to existing node help docs and addition of new ones will trigger downside pipeline for CL bot to make new PRs to add/update localization, pending Dynamo team to approve.

Declarations

Check these if you believe they are true

  • The codebase is in a better state after this PR
  • Is documented according to the standards
  • The level of testing this PR includes is appropriate
  • User facing strings, if any, are extracted into *.resx files
  • All tests pass using the self-service CI.
  • Snapshot of UI changes, if any.
  • Changes to the API follow Semantic Versioning and are documented in the API Changes document.
  • This PR modifies some build requirements and the readme is updated
  • This PR contains no files larger than 50 MB

Release Notes

Enable node help docs source level localization pipeline

Reviewers

@DynamoDS/dynamo

FYIs

@ZbynekHanak-RWS

ZbynekHanak-RWS and others added 14 commits September 19, 2024 11:06
After a several tests I realized that the .resx files with localized strings can be added directly to the project directory location (csproj) level without the need of using Visual Studio. Usually when using Visual Studio for adding resources the <EmbeddedResource> tag is added inside the .csproj file but in this case after testing the localization for other languages was working as expected without using Visual Studio for adding resources.
Also the Access Modifier for resx files localized can have the value "No code generation" and the localization in still working.
Extend file_rules to include all *en-US.resx files under src in the repository.
Add file_rules to include DYN Samples files.
@QilongTang QilongTang added this to the 3.5 milestone Jan 8, 2025
@QilongTang QilongTang changed the title Node Help Docs Localization Pipeline Source-level Localization Pipeline - Node Help Docs Jan 8, 2025
content_owner: N/A
lang: [csy, deu, esp, fra, ita, jpn, kor, plk, ptb, rus, chs, cht]
file_rules:
- doc\distrib\NodeHelpFiles\en-US\*.md, doc\distrib\NodeHelpFiles\%langMsNet%\*.md
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only node help docs source level localization turned on for now

@QilongTang QilongTang changed the title Source-level Localization Pipeline - Node Help Docs DYN-7480 Source-level Localization Pipeline - Node Help Docs Jan 8, 2025
Copy link

@github-actions github-actions bot left a 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-7480

@RobertGlobant20
Copy link
Contributor

image

@RobertGlobant20
Copy link
Contributor

The only change was in ContinuousLocalization.yml?

Copy link

github-actions bot commented Jan 8, 2025

UI Smoke Tests

Test: success. 11 passed, 0 failed.
TestComplete Test Result
Workflow Run: UI Smoke Tests
Check: UI Smoke Tests

@QilongTang
Copy link
Contributor Author

The only change was in ContinuousLocalization.yml?

I think so, the changes on md files contribute most of the file changes

@QilongTang
Copy link
Contributor Author

@QilongTang
Copy link
Contributor Author

QilongTang commented Jan 9, 2025

@QilongTang QilongTang requested a review from zeusongit January 9, 2025 16:07
@QilongTang QilongTang merged commit 8c373bf into master Jan 9, 2025
24 of 28 checks passed
@QilongTang QilongTang deleted the master_cl branch January 9, 2025 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants