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

[i18n:zh_tw] Sync and update translation for zh_TW #11212

Merged
merged 2 commits into from
Nov 14, 2022

Conversation

andy0130tw
Copy link
Contributor

This patch translates essentially all strings to locale zh_TW, with the following topics excluded:

  1. GDPR messages (gdprMessage and gdprMessagePrivacyPolicy)
  2. MetaMask Swaps (keys beginning with "swap")
  3. MetaMetrics (keys beginning with "metametrics") -- I am not confident on translating these messages precisely

In addition, a variety of fixes are applied to improve existing translation strings.

  • Sync all missing strings from en strings at v9.5.7.
  • Update some strings where the corresponding message has changed over time (e.g., readdToken).
  • Fix some inappropriate punctuations (e.g., missing trailing periods).
  • Use more common/agreed terms used in Taiwan (合約、助憶詞、帳戶、函式、...).

I translate these blockchain-related terminologies to the best of my knowledge, but I am open to any adjustments.

@andy0130tw andy0130tw requested a review from a team as a code owner May 30, 2021 16:23
@andy0130tw andy0130tw requested a review from PeterYinusa May 30, 2021 16:23
@andy0130tw andy0130tw force-pushed the update-translation-zh-tw branch from ad1c05a to bcb605e Compare May 31, 2021 08:03
@andy0130tw
Copy link
Contributor Author

andy0130tw commented Jun 2, 2021

I guess I should comment here for the CLA bot to access.

Update: It does not work.

@andy0130tw
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA.

@darkwing darkwing added the area-translation Issues relating to translating the app into various languages. label Nov 6, 2021
@mcmire
Copy link
Contributor

mcmire commented Jan 6, 2022

Hey, sorry it's taken us so long to get back to you. We have a lot of new translations coming in via #13206, so I'll wait until that's merged first.

@mcmire mcmire added the blocked label Jan 6, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2022

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@andy0130tw andy0130tw force-pushed the update-translation-zh-tw branch from 048424f to 7bd805a Compare January 10, 2022 21:02
@kumavis kumavis removed the blocked label Jul 30, 2022
@kumavis kumavis changed the title Sync and update translation for zh_TW [i18n:zh_tw] Sync and update translation for zh_TW Jul 30, 2022
@brad-decker
Copy link
Contributor

@andy0130tw thanks for your patience and apologies for the delayed response. The translation system is now active but we still welcome community translations.If you want to see this landed please rebase and resolve conflicts, then tag me for review. Thank you!

@andy0130tw andy0130tw dismissed a stale review via 9c8a035 November 5, 2022 14:22
@andy0130tw andy0130tw force-pushed the update-translation-zh-tw branch 3 times, most recently from 6cab392 to 1a030be Compare November 5, 2022 15:04
@andy0130tw
Copy link
Contributor Author

I resolved conflicts and rebased my translation against the most recent commit. The PR is ready to be merged (again). Due to the fact that it has been long time since I last worked on this, I patched some small edits I spotted and decided to leave new strings as-is.

Comment on lines 2 to 105
"message": "Incongruent transaction data. Please check the transaction details."
},
"QRHardwarePubkeyAccountOutOfRange": {
"message": "No more accounts. If you would like to access another account unlisted below, please reconnect your hardware wallet and select it."
},
"QRHardwareScanInstructions": {
"message": "Place the QR code in front of your camera. The screen is blurred, but it will not affect the reading."
},
"QRHardwareSignRequestCancel": {
"message": "Reject"
},
"QRHardwareSignRequestDescription": {
"message": "After you’ve signed with your wallet, click on 'Get Signature' to receive the signature"
},
"QRHardwareSignRequestGetSignature": {
"message": "Get Signature"
},
"QRHardwareSignRequestSubtitle": {
"message": "Scan the QR code with your wallet"
},
"QRHardwareSignRequestTitle": {
"message": "Request Signature"
},
"QRHardwareUnknownQRCodeTitle": {
"message": "Error"
},
"QRHardwareUnknownWalletQRCode": {
"message": "Invalid QR code. Please scan the sync QR code of the hardware wallet."
},
"QRHardwareWalletImporterTitle": {
"message": "Scan QR Code"
},
"QRHardwareWalletSteps1Description": {
"message": "Connect an airgapped hardware wallet that communicates through QR-codes. Officially supported airgapped hardware wallets include:"
},
"QRHardwareWalletSteps1Title": {
"message": "QR-based HW Wallet"
},
"QRHardwareWalletSteps2Description": {
"message": "AirGap Vault & Ngrave (Coming Soon)"
},
"SIWEAddressInvalid": {
"message": "The address in the sign-in request does not match the address of the account you are using to sign in."
},
"SIWEDomainInvalidText": {
"message": "The site you're attempting to sign into doesn't match the domain in the request. Proceed with caution."
},
"SIWEDomainInvalidTitle": {
"message": "Deceptive site request."
},
"SIWEDomainWarningBody": {
"message": "The website ($1) is asking you to sign in to the wrong domain. This may be a phishing attack.",
"description": "$1 represents the website domain"
},
"SIWEDomainWarningLabel": {
"message": "Unsafe"
},
"SIWELabelChainID": {
"message": "Chain ID:"
},
"SIWELabelExpirationTime": {
"message": "Expires At:"
},
"SIWELabelIssuedAt": {
"message": "Issued At:"
},
"SIWELabelMessage": {
"message": "Message:"
},
"SIWELabelNonce": {
"message": "Nonce:"
},
"SIWELabelNotBefore": {
"message": "Not Before:"
},
"SIWELabelRequestID": {
"message": "Request ID:"
},
"SIWELabelResources": {
"message": "Resources: $1",
"description": "$1 represents the number of resources"
},
"SIWELabelURI": {
"message": "URI:"
},
"SIWELabelVersion": {
"message": "Version:"
},
"SIWESiteRequestSubtitle": {
"message": "This site is requesting to sign in with"
},
"SIWESiteRequestTitle": {
"message": "Sign-in request"
},
"SIWEWarningSubtitle": {
"message": "To confirm you understand, check:"
},
"SIWEWarningTitle": {
"message": "Are you sure?"
},
Copy link
Contributor

Choose a reason for hiding this comment

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

For any keys that are just using the english message please remove those keys, the system falls back to using the english translation when not present for the language.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK. I will remove these keys and rebase again.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have removed redundant keys accordingly.

@andy0130tw andy0130tw force-pushed the update-translation-zh-tw branch 3 times, most recently from 4ab2667 to e7f53bd Compare November 8, 2022 04:57
Synced from en strings at commit
646dbaa
(unreleased since v10.21.1).
@andy0130tw andy0130tw force-pushed the update-translation-zh-tw branch from e7f53bd to a70c655 Compare November 8, 2022 04:59
@andy0130tw andy0130tw requested review from brad-decker and removed request for PeterYinusa November 11, 2022 10:28
@brad-decker
Copy link
Contributor

Thanks @andy0130tw :D -- approved and will seek an additional approval for merge.

@brad-decker brad-decker merged commit 4937bb4 into MetaMask:develop Nov 14, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Nov 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-translation Issues relating to translating the app into various languages.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants