Skip to content

i18n: extract support page strings#17694

Merged
wackerow merged 78 commits into
devfrom
i18n-support-page
Mar 5, 2026
Merged

i18n: extract support page strings#17694
wackerow merged 78 commits into
devfrom
i18n-support-page

Conversation

@myelinated-wackerow
Copy link
Copy Markdown
Collaborator

@myelinated-wackerow myelinated-wackerow commented Mar 2, 2026

Extends branch

Summary

  • Created page-community-support.json namespace with 39 translation keys for the /community/support/ page
  • Replaced all hardcoded English strings in page.tsx with getTranslations() calls using next-intl
  • Registered /community/support/ prefix in translations.ts namespace mapping (before the /community/ catch-all)

TODO

  • Add non-English translations

Question for @konopkja

The customEventOptions values (eventCategory, eventAction, eventName) are currently hardcoded English strings used as analytics identifiers. Should these remain as-is (since they're tracking keys, not user-facing text), or would you prefer them extracted to the i18n namespace as well?

Test plan

  • Verify page renders identically with translated strings
  • Confirm generateMetadata pulls title/description from namespace
  • Check that /community/support/ resolves to page-community-support namespace

🤖 Generated with Claude Code

Create page-community-support.json namespace and replace hardcoded strings with getTranslations() calls. Register namespace in translations.ts.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 2, 2026

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit ff3c2d5
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/69a79f58de2f080008c3c732
😎 Deploy Preview https://deploy-preview-17694.ethereum.it
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 55 (🔴 down 5 from production)
Accessibility: 94 (🟢 up 1 from production)
Best Practices: 100 (no change from production)
SEO: 99 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot added content 🖋️ This involves copy additions or edits tooling 🔧 Changes related to tooling of the project translation 🌍 This is related to our Translation Program labels Mar 2, 2026
wackerow and others added 18 commits March 2, 2026 21:06
Fix 10 critical issues across 7 languages:
- ar: remove extraneous prefix, fix ecosystem term
- de: fix du/Sie register inconsistency
- hi: standardize Ethereum transliteration
- mr: fix brand misspelling, fix token mistranslation
- pt-br: fix broken double verb in bridge-tokens
- sw: remove extraneous "namba" in layer 2
- ur: fix transliteration typo in meta-description

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
Sanitizer: 58 files cleaned (frontmatter quoting,
inline block components, blank line cleanup).

Review fixes across 8 languages:
- ja, ko: unescape bold markers on ethereum.org link
- ko: fix pooled/liquid staking mistranslation
- de: fix du/Sie register in scams contentPreview
- es: fix tu/usted register in scams contentPreview
- es: fix apostar -> hacer staking de
- it: remove duplicate heading in faq
- bn: standardize Ethereum transliteration
- cs: clean garbled dual-term staking heading

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
warnExposedMdxTags now treats any PascalCase tag as
a safe MDX component instead of maintaining a
hardcoded list. Eliminates 384 false-positive
DocLink warnings. Also expands safe HTML element
list with b, mark, cite, and other standard tags.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: wackerow <54227730+wackerow@users.noreply.github.com>
…13-crowdin-translations

i18n: automated translations - support page MD (all languages)
…37-crowdin-translations

i18n: automated translations - support page JSON (all languages)
@github-actions github-actions Bot added the documentation 📖 Change or add documentation label Mar 4, 2026
@wackerow wackerow marked this pull request as ready for review March 4, 2026 02:04
@wackerow wackerow merged commit 96bce74 into dev Mar 5, 2026
7 checks passed
@wackerow wackerow deleted the i18n-support-page branch March 5, 2026 17:17
@konopkja
Copy link
Copy Markdown
Contributor

konopkja commented Mar 5, 2026

tracking - Keep them as hardcoded English strings.

@konopkja
Copy link
Copy Markdown
Contributor

konopkja commented Mar 5, 2026

as always though im looking at english only

@wackerow wackerow mentioned this pull request Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content 🖋️ This involves copy additions or edits documentation 📖 Change or add documentation tooling 🔧 Changes related to tooling of the project translation 🌍 This is related to our Translation Program

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants