1
0
Fork 0
mirror of https://github.com/maziggy/bambuddy.git synced 2026-05-09 08:25:54 +02:00

[PR #627] [MERGED] Sync i18n locales: add 655 translated keys, remove 127 stale keys #1088

Closed
opened 2026-05-07 00:15:56 +02:00 by BreizhHardware · 0 comments

📋 Pull Request Information

Original PR: https://github.com/maziggy/bambuddy/pull/627
Author: @aneopsy
Created: 3/6/2026
Status: Merged
Merged: 3/6/2026
Merged by: @maziggy

Base: 0.2.2b2Head: fix-i8n


📝 Commits (1)

  • ae7c6b4 Add missing i18n keys and remove stale keys across 5 locales

📊 Changes

5 files changed (+2091 additions, -1005 deletions)

View changed files

📝 frontend/src/i18n/locales/fr.ts (+77 -21)
📝 frontend/src/i18n/locales/it.ts (+693 -26)
📝 frontend/src/i18n/locales/ja.ts (+1102 -931)
📝 frontend/src/i18n/locales/pt-BR.ts (+80 -22)
📝 frontend/src/i18n/locales/zh-CN.ts (+139 -5)

📄 Description

Description

Comparison against EN (3,054 keys) revealed 650+ missing keys across it, pt-BR, fr, ja, and zh-CN, plus 127 stale keys from past refactors. All missing keys were added with proper native translations. Stale keys were cleaned up.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test addition or update

Changes Made

Locale Added Removed
it 485 1
pt-BR 51 10
fr 47 0
ja 47 116
zh-CN 25 1

All new keys include proper translations: Italian, Brazilian Portuguese, French, Japanese, and Simplified Chinese.

Testing

  • I have tested this on my local machine

  • locales.test.ts i18n parity check passes (all locales match EN's 3,054-key structure)

  • All 1058 frontend tests pass (71 test files)

  • npm run build succeeds (tsc + Vite)

  • Spot-checked translations across locales for correctness

Checklist

  • My code follows the project's coding style
  • My changes generate no new warnings
  • I have tested my changes thoroughly

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/maziggy/bambuddy/pull/627 **Author:** [@aneopsy](https://github.com/aneopsy) **Created:** 3/6/2026 **Status:** ✅ Merged **Merged:** 3/6/2026 **Merged by:** [@maziggy](https://github.com/maziggy) **Base:** `0.2.2b2` ← **Head:** `fix-i8n` --- ### 📝 Commits (1) - [`ae7c6b4`](https://github.com/maziggy/bambuddy/commit/ae7c6b4dd2fb97c24d7eca1483f8e40f14ca1634) Add missing i18n keys and remove stale keys across 5 locales ### 📊 Changes **5 files changed** (+2091 additions, -1005 deletions) <details> <summary>View changed files</summary> 📝 `frontend/src/i18n/locales/fr.ts` (+77 -21) 📝 `frontend/src/i18n/locales/it.ts` (+693 -26) 📝 `frontend/src/i18n/locales/ja.ts` (+1102 -931) 📝 `frontend/src/i18n/locales/pt-BR.ts` (+80 -22) 📝 `frontend/src/i18n/locales/zh-CN.ts` (+139 -5) </details> ### 📄 Description ## Description Comparison against EN (3,054 keys) revealed 650+ missing keys across it, pt-BR, fr, ja, and zh-CN, plus 127 stale keys from past refactors. All missing keys were added with proper native translations. Stale keys were cleaned up. ## Type of Change - [x] Bug fix (non-breaking change that fixes an issue) - [ ] New feature (non-breaking change that adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [ ] Documentation update - [ ] Code refactoring - [ ] Performance improvement - [ ] Test addition or update ## Changes Made | Locale | Added | Removed | |--------|-------|---------| | it | 485 | 1 | | pt-BR | 51 | 10 | | fr | 47 | 0 | | ja | 47 | 116 | | zh-CN | 25 | 1 | All new keys include proper translations: Italian, Brazilian Portuguese, French, Japanese, and Simplified Chinese. ## Testing - [x] I have tested this on my local machine - [x] `locales.test.ts` i18n parity check passes (all locales match EN's 3,054-key structure) - [x] All 1058 frontend tests pass (71 test files) - [x] `npm run build` succeeds (tsc + Vite) - [x] Spot-checked translations across locales for correctness ## Checklist - [x] My code follows the project's coding style - [x] My changes generate no new warnings - [x] I have tested my changes thoroughly --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
BreizhHardware 2026-05-07 00:15:56 +02:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/bambuddy-maziggy-1#1088
No description provided.