Skip to content

Clean up i18n namespaces and localize LN invoice copy feedback - #50

Merged
pepakriz merged 1 commit into
mainfrom
codex/review-i18next-translations-for-usage
Mar 30, 2026
Merged

Clean up i18n namespaces and localize LN invoice copy feedback#50
pepakriz merged 1 commit into
mainfrom
codex/review-i18next-translations-for-usage

Conversation

@pepakriz

Copy link
Copy Markdown
Contributor

Motivation

  • Align registered i18n namespaces with the actual locale modules to avoid stale/unused namespace entries.
  • Replace a hardcoded English toast message with a translation key so the LN-copy feedback is localizable.
  • Remove an unused locale file that was no longer referenced by the resources wiring.

Description

  • Removed the obsolete clients namespace and added the missing transactions namespace in lib/i18n/config.ts to match loaded resources.
  • Replaced the hardcoded copy-confirmation string in components/receive-payment.tsx with t("payments:detail.messages.ln-invoice-copied-to-clipboard").
  • Added the new translation key payments:detail.messages.ln-invoice-copied-to-clipboard to locales/en/payments.ts and locales/cs/payments.ts.
  • Deleted the unused file locales/en/clients.ts as it was not referenced by the resources setup.

Testing

  • Ran a parity check script comparing locales/en and locales/cs which reported no unmatched files (onlyEn [], onlyCs []).
  • Ran bun run check:lint which completed successfully (existing warnings remain but no new lint errors introduced).
  • Ran type checks with bun run check:types (tsc) which failed due to pre-existing TypeScript errors in app/(client)/settings/page.tsx unrelated to these changes.

Codex Task

@pepakriz
pepakriz merged commit 48aced0 into main Mar 30, 2026
2 checks passed
@pepakriz
pepakriz deleted the codex/review-i18next-translations-for-usage branch March 30, 2026 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant