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

🌐 Translations - improved translation for zh cn 20241108 #679

Open
wants to merge 38 commits into
base: support/3.2
Choose a base branch
from

Conversation

purplegrape
Copy link
Contributor

@purplegrape purplegrape commented Nov 8, 2024

improved translation for zh cn 20241108

this pr is for zh-cn entries in "web/dictionaries" and "web/datamodels"

1.fix some missing line (compared to en.dict )
2. minor translation improvement

@purplegrape
Copy link
Contributor Author

The structure of entries in datamodels change a lot, just do as en.dict

Copy link
Contributor

@Hipska Hipska left a comment

Choose a reason for hiding this comment

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

I have some objections regarding the file headings..

*
*/
/**
* @author Robert Deng <[email protected]>
Copy link
Contributor

Choose a reason for hiding this comment

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

Not nice to remove original authors. Same remark for the other files where You removed the @author.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Robert Deng is the zh cn maintainer quite many years ago,I am the main contributor of zh-cn in the recent 2 years
in my mind,the “@author” line was removed by me since itop 2.4 or 2.5
why it appears in iTop 3.2 again?

Copy link
Contributor

Choose a reason for hiding this comment

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

He still is the original author. Should not be removed, only add a new @author if needed.

Comment on lines 20 to 51
'Class:UserLocal/Attribute:expiration/Value:force_expire' => '已过期',
'Class:UserLocal/Attribute:expiration/Value:force_expire+' => '',
'Class:UserLocal/Attribute:expiration/Value:never_expire' => '永不过期',
'Class:UserLocal/Attribute:expiration/Value:never_expire+' => '',
'Class:UserLocal/Attribute:expiration/Value:force_expire' => '已过期',
'Class:UserLocal/Attribute:expiration/Value:force_expire+' => '',
Copy link
Contributor

Choose a reason for hiding this comment

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

Aren't these items supposed to be sorted alphabetically?

Dict::Add('ZH CN', 'Chinese', '简体中文', [
'theme:darkmoon' => '暗月',
]);
]);
Copy link
Contributor

Choose a reason for hiding this comment

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

Please re-add the missing newline at end of file.

Comment on lines 3 to 6
* Localized data
* Copyright (C) 2013-2024 Combodo SAS
*
* @copyright Copyright (C) 2010-2024 Combodo SAS
* @license https://opensource.org/licenses/AGPL-3.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Previous way was more correct, please revert this.

Copy link
Contributor

@Hipska Hipska left a comment

Choose a reason for hiding this comment

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

Still some to fix..

Note I didn't put suggestions for every file, but I think you get it, please apply on all files where necessary.

@purplegrape
Copy link
Contributor Author

Still some to fix..

Note I didn't put suggestions for every file, but I think you get it, please apply on all files where necessary.

In your words,there is too much work to do
I've checked line by line against en.dict
I think it's fine to overwrite the files directly if no php syntax error

@purplegrape purplegrape changed the base branch from develop to support/3.2 November 8, 2024 10:44
@purplegrape
Copy link
Contributor Author

fixed, this pr should merge to support/3.2 , no conflicts now

@jf-cbd
Copy link
Contributor

jf-cbd commented Nov 13, 2024

Hello, thanks a lot @purplegrape for your contributions and editions :D
I will perform the final checks and make any necessary edits.

@Hipska
Copy link
Contributor

Hipska commented Nov 13, 2024

@purplegrape The current state of your PR is also undoing all the fixes in EN US dicts done by #678

@jf-cbd
Copy link
Contributor

jf-cbd commented Nov 13, 2024

@purplegrape The current state of your PR is also undoing all the fixes in EN US dicts done by #678

True, but the current PR was created before I merged #678, so I'll address this.

@Hipska
Copy link
Contributor

Hipska commented Nov 13, 2024

That should not happen by default. Its the way @purplegrape did a rebase and then copied over his old files or something..

@jf-cbd jf-cbd self-requested a review November 13, 2024 08:59
jf-cbd and others added 8 commits November 13, 2024 11:20
…pe/support/3.2

# Conflicts:
#	datamodels/2.x/itop-incident-mgmt-itil/dictionaries/en.dict.itop-incident-mgmt-itil.php
#	dictionaries/zh_cn.dictionary.itop.core.php
Copy link
Contributor

@Hipska Hipska left a comment

Choose a reason for hiding this comment

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

New batch of changes to apply

jf-cbd and others added 12 commits November 13, 2024 12:05
…top-change-mgmt-itil.php

Co-authored-by: Thomas Casteleyn <[email protected]>
…itop-files-information.php

Co-authored-by: Thomas Casteleyn <[email protected]>
….itop-incident-mgmt-itil.php

Co-authored-by: Thomas Casteleyn <[email protected]>
…op-knownerror-mgmt.php

Co-authored-by: Thomas Casteleyn <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: First review needed
Development

Successfully merging this pull request may close these issues.

4 participants