-
Notifications
You must be signed in to change notification settings - Fork 5.5k
chore: New Crowdin Translations by GitHub Action #38977
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
base: main
Are you sure you want to change the base?
Conversation
| }, | ||
| "alert": { | ||
| "message": "Warnhinweis" | ||
| }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Translation key removed while still in use
The alertAccountTypeUpgradeMessage translation key is being removed from 14 locale files (de, el, es, fr, hi, id, ja, ko, pt, ru, tl, tr, vi, zh_CN), but this key is still actively used in AccountTypeMessage.tsx and remains defined in the English source file. Users with these language settings will see missing translations or English fallback text when upgrading to a smart account.
Additional Locations (2)
395dc0d to
8f5b887
Compare
| }, | ||
| "alert": { | ||
| "message": "Warnhinweis" | ||
| }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Translations removed for active message key
The alertAccountTypeUpgradeMessage translation key is being removed from 14 locale files (de, el, es, fr, hi, id, ja, ko, pt, ru, tl, tr, vi, zh_CN), but this key is still actively used in AccountTypeMessage.tsx and present in the source English locale file. Users in affected locales will see English fallback text instead of their native language translations when viewing the account type upgrade alert.
Additional Locations (2)
8f5b887 to
036496a
Compare
| "message": "AirGap Vault" | ||
| }, | ||
| "alert": { | ||
| "message": "Warnhinweis" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Active translation key removed from multiple locale files
The alertAccountTypeUpgradeMessage translation key is being removed from 14 locale files (de, el, es, fr, hi, id, ja, ko, pt, ru, tl, tr, vi, zh_CN), but this key is actively used in AccountTypeMessage.tsx to display the smart account upgrade message. Users in these locales will see either the English fallback text or the raw key name instead of their translated content. The English source file still contains this key, suggesting this is an unintended Crowdin sync issue.
Additional Locations (2)
036496a to
222bab9
Compare
| "message": "AirGap Vault" | ||
| }, | ||
| "alert": { | ||
| "message": "Warnhinweis" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Translations removed for actively used translation key
The alertAccountTypeUpgradeMessage translation key is being removed from 14 locale files (de, el, es, fr, hi, id, ja, ko, pt, ru, tl, tr, vi, zh_CN), but this key is actively used in AccountTypeMessage.tsx. The English locales still contain this key. Users with these locales will see either English fallback text or potentially the raw key string instead of their localized message when viewing account upgrade alerts.
Additional Locations (2)
222bab9 to
260a9a5
Compare
| "alert": { | ||
| "message": "Warnhinweis" | ||
| }, | ||
| "alertAccountTypeUpgradeMessage": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Translations removed for actively used localization key
The alertAccountTypeUpgradeMessage translation key is being removed from 14 locale files, but the key is still defined in the English source locale and actively used in AccountTypeMessage.tsx. This will cause users with German, Greek, Spanish, French, Hindi, Indonesian, Japanese, Korean, Portuguese, Russian, Tagalog, Turkish, Vietnamese, and Chinese language settings to see English fallback text instead of their localized translation when encountering the smart account upgrade alert.
Additional Locations (2)
Builds ready [260a9a5]
UI Startup Metrics (1302 ± 106 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
260a9a5 to
461ed38
Compare
| "alert": { | ||
| "message": "Warnhinweis" | ||
| }, | ||
| "alertAccountTypeUpgradeMessage": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Translation key removed while still in use
The alertAccountTypeUpgradeMessage key is being removed from 14 non-English locales, but the key still exists in the English locale and is actively used in AccountTypeMessage.tsx. This will cause users with affected locales (German, Greek, Spanish, French, Hindi, Indonesian, Japanese, Korean, Portuguese, Russian, Tagalog, Turkish, Vietnamese, Chinese) to see the English fallback message instead of their localized translation when viewing the account type upgrade alert. This appears to be an unintended Crowdin sync issue.
Additional Locations (2)
461ed38 to
e69cdbc
Compare
Builds ready [e69cdbc]
UI Startup Metrics (1285 ± 100 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
e69cdbc to
0feb2f3
Compare
| "alert": { | ||
| "message": "Warnhinweis" | ||
| }, | ||
| "alertAccountTypeUpgradeMessage": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Translation key removed but still used in code
The alertAccountTypeUpgradeMessage translation key is being removed from 14 non-English locale files (de, el, es, fr, hi, id, ja, ko, pt, ru, tl, tr, vi, zh_CN), but the key still exists in app/_locales/en/messages.json and is actively used in AccountTypeMessage.tsx. This causes a translation regression where users with these locale settings will see the English fallback message instead of their localized translation when viewing the smart account upgrade alert.
Additional Locations (2)
0feb2f3 to
87db67f
Compare
87db67f to
188a1c7
Compare
Builds ready [da5511b]
UI Startup Metrics (1274 ± 110 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
da5511b to
0c92edf
Compare
Builds ready [0c92edf]
UI Startup Metrics (1328 ± 92 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
0c92edf to
b523f21
Compare
Builds ready [b523f21]
UI Startup Metrics (1264 ± 107 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
b523f21 to
59013c8
Compare
Builds ready [59013c8]
UI Startup Metrics (1253 ± 94 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
59013c8 to
bdbef58
Compare
Builds ready [bdbef58]
UI Startup Metrics (1229 ± 93 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
bdbef58 to
8b8c230
Compare
| "alert": { | ||
| "message": "Warnhinweis" | ||
| }, | ||
| "alertAccountTypeUpgradeMessage": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Translations removed for actively used message key
The alertAccountTypeUpgradeMessage translation key is removed from 14 locale files (de, el, es, fr, hi, id, ja, ko, pt, ru, tl, tr, vi, zh_CN), but this key is still actively used in AccountTypeMessage.tsx and exists in the English source file. Users with these language settings will see either the English fallback text or a broken translation when encountering the account upgrade alert. This appears to be an erroneous removal from the Crowdin sync.
Additional Locations (2)
Builds ready [8b8c230]
UI Startup Metrics (1280 ± 116 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
8b8c230 to
d3a1091
Compare
| "alert": { | ||
| "message": "Warnhinweis" | ||
| }, | ||
| "alertAccountTypeUpgradeMessage": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Translations removed for key still in use
The alertAccountTypeUpgradeMessage translation key is being removed from 14 locales (de, el, es, fr, hi, id, ja, ko, pt, ru, tl, tr, vi, zh_CN), but it is still present in the English source file (en/messages.json) and actively used in AccountTypeMessage.tsx. Users with these language settings will see untranslated English text for the smart account upgrade alert instead of their localized translation.
Additional Locations (2)
Builds ready [d3a1091]
UI Startup Metrics (1289 ± 95 ms)
📊 Page Load Benchmark ResultsCurrent Commit: 📄 Localhost MetaMask Test DappSamples: 100 Summary
📈 Detailed Results
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Note
Cleans up i18n resources by deleting the obsolete
alertAccountTypeUpgradeMessageentry across locales.alertAccountTypeUpgradeMessagefrommessages.jsonin:de,el,es,fr,hi,id,ja,ko,pt,ru,tl,tr,vi,zh_CNWritten by Cursor Bugbot for commit d3a1091. This will update automatically on new commits. Configure here.