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

Request to add simplified Chinese Translation #319

Merged
merged 14 commits into from
Jun 8, 2022

Conversation

Augustakit
Copy link
Contributor

I updated the simplified Chinese translation a few days ago. Please merge it

Simplified Chinese language added
Simplified Chinese language added
Simplified Chinese language added
Simplified Chinese language added
Copy link
Collaborator

@diwako diwako left a comment

Choose a reason for hiding this comment

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

Make sure to leave entries of other languages.

addons/main/stringtable.xml Show resolved Hide resolved
addons/main/stringtable.xml Show resolved Hide resolved
@diwako
Copy link
Collaborator

diwako commented Apr 20, 2022

Reported issues from the linting

Checking addons/main/stringtable.xml:
  ERROR: Incorrect number of indenting spaces on line 71, currently 13, should be 12.
  ERROR: Incorrect number of indenting spaces on line 89, currently 13, should be 12.
  ERROR: Incorrect number of indenting spaces on line 143, currently 13, should be 12.
  ERROR: Incorrect number of indenting spaces on line 195, currently 13, should be 12.
  ERROR: Incorrect number of indenting spaces on line 204, currently 13, should be 12.
  ERROR: Incorrect number of indenting spaces on line 213, currently 13, should be 12.
addons/danger/stringtable.xml:
	396: Trailing whitespace
addons/main/stringtable.xml:
	71: Wrong amount of left-padding spaces(want multiple of 4)
	89: Wrong amount of left-padding spaces(want multiple of 4)
	143: Wrong amount of left-padding spaces(want multiple of 4)
	195: Wrong amount of left-padding spaces(want multiple of 4)
	204: Wrong amount of left-padding spaces(want multiple of 4)
	213: Wrong amount of left-padding spaces(want multiple of 4)
addons/wp/stringtable.xml:
	192: Trailing whitespace
	686: Trailing whitespace
	819: Wrong amount of left-padding spaces(want multiple of 4)
	1221: Trailing whitespace

Fix the error diwako corrected
Fix the error diwako corrected
Fix the error diwako corrected
@Augustakit
Copy link
Contributor Author

I have fixed all the problems you pointed out. Please help me check them. Thank you very much

Fix the error diwako corrected
Fix the error diwako corrected
Fix the error diwako corrected
@@ -255,6 +286,7 @@
<Polish>Promień</Polish>
<Russian>Радиус</Russian>
<Italian>Raggio</Italian>
<Chinesesimp>建筑和静态武器半径范围内驻守并配备/n两倍于该数字的人员用于巡逻</Chinesesimp>
Copy link
Collaborator

Choose a reason for hiding this comment

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

I only google translated this to check as this looked rather long. It seems this is the tooltip of "TaskCamp"_Radius and not the display name.

The following entries are also not correct as they represent the next entry in the string table.

Correct some language errors
Correct some language errors
Correct some language errors
@Augustakit
Copy link
Contributor Author

Sorry. I'm so very sorry about that one.I accidentally uploaded some translations at the wrong time, which led you to see such results, but I have corrected them now, and checked the pages of other translation documents and corrected them again.

@jokoho48 jokoho48 added this to the 2.5.4 milestone Jun 8, 2022
@diwako diwako merged commit bb14bfa into nk3nny:master Jun 8, 2022
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.

3 participants