i18n: Gemini translations (de)#17842
Conversation
✅ Deploy Preview for ethereumorg ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
github-actions
Bot
added
content 🖋️
|
🌐 Translation review started. View progress |
Translation Quality ReviewPR: #17842 | Language: de (German) | Files: 304 | Date: 2026-03-24 | Score: 8.6/10
Critical Issues (9 total):
Warnings:
Strengths: Zero PoS/PoW inversions. State correctly rendered as Zustand throughout. All brand names preserved. No cross-script contamination. No MDX syntax issues. All internal hrefs correct. Excellent glossary compliance. Reviewed by Claude Code -- 5 parallel agents, 304 files To apply fixes, reply with: |
Run post_import_sanitize on German translations: - Fix smart quotes in JSX tag attributes - Restore brand names (Alchemy, Hardhat, etc.) - Sync protected frontmatter fields from English - Quote frontmatter values with unsafe YAML chars - Remove orphaned closing HTML tags - Fix missing link brackets and Crowdin manglings - Restore blank lines lost in translation Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
- EN: add missing space after Emoji tag - DE: translate "Fun Fact!" to "Wissenswertes!" - Sanitizer: skip <https://...> URL autolinks in exposed MDX tag warnings (valid markdown syntax) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Add fixLowercasedMdxComponents to restore PascalCase component names from English source. Also suppress false-positive warnings for <https://...> URL autolinks and reorder warnExposedMdxTags to run after the fix. 7 new tests, 480 total passing. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Sanitizer auto-fixes for DE translations: - ethereum-forks: <emoji> -> <Emoji> (PascalCase) - payments: normalize ButtonLink indentation Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
[Ethereum]() -> [Ethereum](/) in both EN source and DE translation. The empty href was a pre-existing bug. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
github-actions
Bot
added
documentation 📖
Remove unused _fullName capture group in fixLowercasedMdxComponents regex. Document the unused-imports/no-unused-vars ESLint rule in CLAUDE.md to prevent future build failures. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
removeOrphanedClosingTags was stripping valid </em> when <em> was on the previous line. Track unclosed openers across lines to prevent false orphan removal. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Sanitizer auto-fix for ethereum-forks and payments: - Restore </em> before </li> (cross-line em tags) - Normalize ButtonLink indentation Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Translation Quality Review -- UpdatedPR: #17842 | Language: de (German) | Files: 304 | Date: 2026-03-25 | Score: 8.9/10 (up from 8.6) Critical Issues Resolved
Critical Issues Remaining (2)
Warnings (unchanged)
Sanitizer Improvements (shipped with this PR)
Reviewed by Claude Opus 4.6 |
Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
myelinated-wackerow
left a comment
myelinated-wackerow
left a comment
There was a problem hiding this comment.
Approved
All critical issues from the initial review have been addressed:
- Sanitizer auto-fixes applied (brand names, smart quotes, YAML quoting, frontmatter sync -- 45 files)
- MDX build failures fixed (missing
</em>, lowercase<Emoji>component) - Inherited EN bug fixed (
[Ethereum]()empty href in social-networks, PoS linking to/pow/) - "Fun Fact!" translated to "Wissenswertes!"
Note: The clients -> Anwendungen translation throughout the roadmap pages is consistent with the current Crowdin glossary entry (client -> Anwendung). The compound terms (consensus client -> Konsens-Client, execution client -> Ausführungs-Client) correctly preserve "Client". The standalone glossary entry may warrant a future cleanup to align with the compound terms, but this is not a blocker for this PR.
Sanitizer improvements shipped with this PR:
fixLowercasedMdxComponents(7 tests)removeOrphanedClosingTagscross-line tracking fix (1 test)warnExposedMdxTagsURL autolink false positive suppression
481 sanitizer tests passing.
Reviewed by Claude Opus 4.6
|
Noting for follow-up |
2 participants
Description
This PR contains automated Gemini translations via Gemini.
🔗 View workflow run
Languages translated
de
JSON changes (
src/intl/{locale}/)Markdown changes (
public/content/translations/{locale}/)