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

Merge master_cl-localization into master_cl #15866

Merged
merged 1 commit into from
Mar 4, 2025

Conversation

escbuild
Copy link
Collaborator

@escbuild escbuild commented Feb 27, 2025

Hi , could you please review and merge this PR?

================== Word Count Statistics begin ==================

GMT Tue Mar 4 18:31:05 2025: Localization Engineer Lead for this PR is [email protected],[email protected].

Vendor Engineer for this PR is [email protected].

Localization branch is updated.
New/Updated strings status
CHS: 61 strings (606 words) are in English
CHT: 61 strings (606 words) are in English
CSY: 62 strings (607 words) are in English
DEU: 61 strings (606 words) are in English
ESP: 61 strings (606 words) are in English
FRA: 61 strings (606 words) are in English
ITA: 61 strings (606 words) are in English
JPN: 61 strings (606 words) are in English
KOR: 61 strings (606 words) are in English
PLK: 61 strings (606 words) are in English
PTB: 62 strings (607 words) are in English
RUS: 61 strings (606 words) are in English

Please note that If the word count above shows some words in English, it is because the localization team hasn't processed all the new/updated strings yet. These words will be translated in the next translation loop.

Please merge this PR anyway as it will align the localizable file(s) content with the English file(s) content and prevent any functionality issue caused from misaligned localized file(s) versus English file(s).

If you are about to release and you are concerned about these strings showing up in English, please contact your localization contact/project team. Thanks.

=================== Word Count Statistics end ==================

Copy link
Contributor

@RobertGlobant20 RobertGlobant20 left a comment

Choose a reason for hiding this comment

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

I can see that most of the html files are not translated to the specified languages (like previous PR), is this expected?
Other than this the changes look good to me.

From build https://ls.jenkins.autodesk.com/job/CL/job/DynamoCore_CL/211/
Professional translation: all languages
Pseudo translation: none
@escbuild escbuild force-pushed the master_cl-localization branch from 892c520 to ef48dd0 Compare March 4, 2025 18:31
@escbuild
Copy link
Collaborator Author

escbuild commented Mar 4, 2025

This is a friendly reminder.

Hi @zeusongit, @QilongTang, @reddyashish,
This PR has been opened more than 2 days ago. Could you please merge it?

Thanks.

@QilongTang
Copy link
Contributor

I can see that most of the html files are not translated to the specified languages (like previous PR), is this expected? Other than this the changes look good to me.

Thanks for the notice! I think the localization team tend to provide English content first and provide localized content when the actual effort included in their next dev cycle.

@QilongTang QilongTang merged commit 4232512 into master_cl Mar 4, 2025
14 of 18 checks passed
@QilongTang QilongTang deleted the master_cl-localization branch March 4, 2025 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants