i18n: automated translation roadmap/glamsterdam (remaining languages)#17770
Conversation
✅ Deploy Preview for ethereumorg ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
🌐 Translation review started. View progress |
Translation Quality ReviewPR: #17770 Summary by Language
Average Quality Score: 8.9/10 Critical Issues (Must Fix)ar (Arabic)
pt-br (Brazilian Portuguese)
Warnings (Should Review)ar (Arabic)
hi (Hindi)
id (Indonesian)
it (Italian)
ko (Korean)
pt-br (Brazilian Portuguese)
vi (Vietnamese)No warnings -- clean translation. Quality Scores by Languagear (Arabic) - 8.8/10
Strong translation with consistent terminology. Two link issues (broken syntax line 245, translated URL line 258) must be fixed. hi (Hindi) - 8.6/10
Good translation overall. Minor brand name and transliteration consistency issues. id (Indonesian) - 7.8/10
Lowest-scoring translation. "Scale" → "Peningkatan" and inconsistent "staking" terminology need attention. it (Italian) - 9.6/10
Highest-scoring translation. Production-ready with only minor stylistic notes. ko (Korean) - 9.0/10
High quality but has formatting bugs: broken link (line 258) and escaped bold markers throughout that will affect rendering. pt-br (Brazilian Portuguese) - 9.1/10
Strong translation. Translated URL vi (Vietnamese) - 9.2/10
Clean translation with no issues. Ready to merge. To apply fixes, reply with: Reviewed by Claude Code |
…T20-36-03-crowdin-translations
- fixEscapedBoldAndItalic: add non-ASCII support to regex lookbehind/lookahead; use <strong>/<em> tags when followed by non-ASCII chars (CommonMark compat) - fixMissingLinkBrackets: new function to restore missing [] around link text using English source comparison (handles both text(/path/) and text/path/) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
ar: restore missing link brackets on /roadmap/ link,
add space before heading anchor
ko: unescape 8 bold markers to <strong> tags
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
ar: /security/ href, GitHub casing pt-br: /security/ href Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Translation Quality ReviewQuality Scores (post-fix)
Issues Found & Fixed
Remaining Warnings (non-blocking)
Sanitizer ImprovementsTwo sanitizer bugs were fixed and applied to this PR:
All 7 critical issues resolved. All languages ready to ship. Reviewed by Claude Opus 4.6 |
Crowdin pulled AlertTitle text outside the Alert component as bold markdown. Moved translated title text back inside AlertTitle tags to match English MDX structure. Full structural audit confirmed all 24 translations now match English component nesting. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Match current English: fix stale bullets, split merged paragraph into 3, add missing MEV-Boost paragraph, fix "closed-source" -> "third-party middleware". Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Add missing "Track changes" (page-roadmap-release-forkcast) translation to es, zh, ar, hi, id, and it page-roadmap namespace files. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
|
@khalidbalosh211-ux need something? |
|
Cerberus, block khalidbalosh211-ux |
|
|
Add fixSmartQuotesInJsxAttributes to replace curly quotes (U+201C-201F) with straight ASCII quotes inside JSX component attribute values. 7 new tests, 252 total passing. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Fix curly quotes in <YouTube> tag attributes for cs, de, pl, zh glamsterdam translations. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
myelinated-wackerow
left a comment
There was a problem hiding this comment.
Approval Review
LGTM -- build passes, all 24 glamsterdam translations verified against English source.
Post-import fixes applied (8 commits)
| Commit | Scope | What |
|---|---|---|
d6ede0f |
sanitizer | Fix escaped bold/italic for non-Latin scripts; add fixMissingLinkBrackets |
b89b4cc |
ar, ko | Sanitizer fixes for escaped bold + missing link brackets |
0c6ec2d |
ar, pt-br | Restore translated hrefs (/seguranca/ -> /security/), fix GitHub casing |
f2230306 |
17 langs | Restore <AlertTitle> in update Alert (was missing from older translations) |
98cfda6 |
17 langs | Update stale ePBS section content to match current English (split merged paragraphs, add missing MEV-Boost paragraph, fix bullet points) |
e36a979 |
6 langs | Add missing forkcast translation key to page-roadmap JSON |
42a686c |
sanitizer | New fixSmartQuotesInJsxAttributes -- catches curly quotes in JSX tag attributes (7 tests, 252 total passing) |
4fc7c9e |
cs, de, pl, zh | Apply smart-quote fix to 4 glamsterdam files that broke MDX build |
Quality check summary
- Structural parity: All 24 translations match English paragraph/bullet/heading counts
- AlertTitle: Present in update Alert, absent from info Alert -- all 24 clean
- Hrefs: No translated hrefs remaining
- JSX attributes: No smart quotes in component tags
- Build: Passes for all 24 locales
Known non-blocking notes
- Some older translations (the 17 pre-existing) have minor prose-level translation quality variance vs current English outside the ePBS section (grammar artifacts from Crowdin MT). Not blocking -- these are cosmetic and can be addressed in a follow-up Crowdin review cycle.
- CJK files (ja, zh, zh-tw, ko) use
<strong>/<em>HTML tags instead of**/*markdown for CommonMark compatibility with non-Latin scripts. This is correct behavior.
---Reviewed by Claude Opus 4.6

Description
This PR contains automated gemini-2.5-pro translations from Crowdin.
🔗 View workflow run
Languages translated
ar, hi, id, it, ko, pt-br, vi
Markdown changes (
public/content/translations/{locale}/)