Skip to content

[hot-fix] patch(intl): sim component namespace to /wallets#18000

Merged
pettinarip merged 1 commit into
masterfrom
patch-i18n-ns
Apr 21, 2026
Merged

[hot-fix] patch(intl): sim component namespace to /wallets#18000
pettinarip merged 1 commit into
masterfrom
patch-i18n-ns

Conversation

@myelinated-wackerow
Copy link
Copy Markdown
Collaborator

@myelinated-wackerow myelinated-wackerow commented Apr 20, 2026

Description

Fixes broken translation keys (sim-ca-title, sim-sr-title, sim-cw-title, etc.) rendering on the /wallets/ page where the Wallet Simulator is embedded.

Root cause

app/[locale]/wallets/page.tsx uses getRequiredNamespacesForPage("/wallets") to pick the messages passed into I18nProvider. That helper returned page-wallets (plus common, glossary, quizzes) but not component-wallet-simulator, so the simulator's strings were missing from the client bundle and fell back to raw keys.

The homepage was unaffected because app/[locale]/page.tsx explicitly picks component-wallet-simulator alongside page-index.

Fix

Added "/wallets/": ["component-wallet-simulator"] to EXACT_PATH_ADDITIONAL_NAMESPACES in src/lib/utils/translations.ts so the namespace is included wherever getRequiredNamespacesForPage is called for that route.

Testing

  • Visit /wallets/ (English) and confirm the simulator renders translated strings rather than raw keys.

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

netlify Bot commented Apr 20, 2026

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 18b0082
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/69e6bcf86ec343000822ef0b
😎 Deploy Preview https://deploy-preview-18000.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: 54 (🔴 down 1 from production)
Accessibility: 93 (no change from production)
Best Practices: 100 (🟢 up 1 from production)
SEO: 98 (🔴 down 1 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 the tooling 🔧 Changes related to tooling of the project label Apr 20, 2026
@pettinarip pettinarip merged commit a0b54c6 into master Apr 21, 2026
15 checks passed
@pettinarip pettinarip deleted the patch-i18n-ns branch April 21, 2026 08:11
@wackerow wackerow mentioned this pull request Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tooling 🔧 Changes related to tooling of the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants