Skip to content

✨ Localize transactional emails - #192

Merged
Logan2234 merged 8 commits into
mainfrom
codex/transactional-email-i18n
Sep 5, 2026
Merged

✨ Localize transactional emails#192
Logan2234 merged 8 commits into
mainfrom
codex/transactional-email-i18n

Conversation

@Logan2234

Copy link
Copy Markdown
Owner

Summary

  • localize all transactional email templates in French and English
  • send the signup page locale explicitly and keep Accept-Language as a legacy fallback
  • propagate each recipient's stored locale through every email producer
  • keep newsletter and moderation editorial content exactly as authored
  • add an FR/EN selector to the admin email preview and test-send gallery
  • add localized dates, HTML language metadata, and safer dynamic HTML escaping

Verification

  • API email suite: 21/21 tests passed
  • API full suite: 1,028/1,029 tests passed; the only timeout passed when rerun alone
  • Web full suite: 123/124 tests passed; the only timeout's two cases passed when rerun alone
  • Web production build passed
  • Shared package build passed
  • API production build is currently blocked by the existing Fastify 5.12.1/5.12.3 type mismatch in src/main.ts

Scope

This PR covers emails only. In-app and push notification localization remains a separate follow-up.

Feedback: https://feedback.loomkeep.app/admin/feedback?sort=newest&status=%5B%22open%22%5D&post=post_01m11a5ecdecw9tbwp2686s1rj

@github-actions github-actions Bot added api apps/api (NestJS) web apps/web (SvelteKit) shared packages/shared labels Sep 5, 2026
Comment thread packages/shared/src/dto/auth.ts Fixed
Comment thread apps/api/src/auth/auth.service.spec.ts Fixed
Comment thread apps/api/src/mail/mail.i18n.ts Fixed
Comment thread apps/api/src/mail/mail.service.spec.ts Fixed
Comment thread apps/api/src/mail/mail.service.spec.ts Fixed
Comment thread apps/api/src/mail/mail.service.spec.ts Fixed
Comment thread apps/api/src/mail/mail.service.ts Fixed
Comment thread apps/api/src/mail/mail.service.ts Fixed
Comment thread apps/api/src/reports/report.service.spec.ts Fixed
Comment thread apps/api/src/reports/report.service.spec.ts Fixed
Comment thread apps/api/src/reports/report.service.spec.ts Fixed
@Logan2234
Logan2234 merged commit 1eb229d into main Sep 5, 2026
19 of 20 checks passed
@Logan2234
Logan2234 deleted the codex/transactional-email-i18n branch September 5, 2026 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api apps/api (NestJS) shared packages/shared web apps/web (SvelteKit)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants