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

[Add] simplified Chinese translation #24

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

julienemo
Copy link

➕ locales for simplified Chinese
ℹ️ the URLs provided in Chinese translation are those of English version

🤷‍♀️ wanted to update the carbon intensity factor for China but didn't find convincing statistics

Copy link

@WalterRobin WalterRobin left a comment

Choose a reason for hiding this comment

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

Another Chinese reviewer proposed the following corrections, what do you think?

{

"moreInformation": {

"message": "More information",

"description": "More information link title"更多链接

},

"moreInformationUrl": {

"message": "https://theshiftproject.org/en/carbonalyser-browser-extension/",

"description": "More information url"更多链接

},

"legalNotice": {

"message": "Legal notice",

"description": "Legal notice link title"法律声明标题链接

},

"legalNoticeUrl": {

"message": "https://theshiftproject.org/en/general-conditions-of-use/",

"description": "Legal notice url”法律声明链接

},

"start": {

"message": "Run analysis”,开始分析

"description": "Run analysis button" 开始分析按钮

},

"stop": {

"message": "Stop analysis”,停止分析

"description": "Stop analysis button"停止分析按钮

},

"reset": {

"message": "Reset data”,重置数据

"description": "Reset data button”数据重置按钮

"topTraffic": {

"message": "Top 5 traffic related to your browsing”,在您浏览的网站中,使用流量最大的五个网站

"description": "Top traffic list title”流量排名列表

"selectRegionPlaceholder": {

"message": "Select your location”选择您的位置

"chargedSmartphonesUnit": {

"message": "charged smartphones" 充满电的智能手机

@julienemo
Copy link
Author

@WalterRobin Thanks for the info and sorry for the late reply

PR updated for all suggestions except for the follows, where I personnally prefer keeping my translation.

moreInformation, moreInformationUrl, legalNotice and legalNoticeUrl

@WalterRobin
Copy link

@julienemo Ok great! Thank you for your help on the translation it's truly appreciated. I've approved your commit

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.

2 participants