[PR #4154] [MERGED] 🌍 improve translation strings - part 2 #5219

Closed
opened 2026-02-28 21:08:23 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/4154
Author: @matt-fidd
Created: 1/14/2025
Status: Merged
Merged: 1/21/2025
Merged by: @matt-fidd

Base: masterHead: translations-2


📝 Commits (10+)

📊 Changes

45 files changed (+305 additions, -186 deletions)

View changed files

📝 packages/desktop-client/e2e/settings.mobile.test.js-snapshots/Mobile-Settings-checks-that-settings-page-can-be-opened-1-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/settings.mobile.test.js-snapshots/Mobile-Settings-checks-that-settings-page-can-be-opened-2-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/settings.mobile.test.js-snapshots/Mobile-Settings-checks-that-settings-page-can-be-opened-3-chromium-linux.png (+0 -0)
📝 packages/desktop-client/src/components/App.tsx (+2 -2)
📝 packages/desktop-client/src/components/accounts/Account.tsx (+2 -2)
📝 packages/desktop-client/src/components/accounts/Reconcile.tsx (+10 -5)
📝 packages/desktop-client/src/components/autocomplete/PayeeAutocomplete.tsx (+1 -1)
📝 packages/desktop-client/src/components/budget/BalanceWithCarryover.tsx (+67 -26)
📝 packages/desktop-client/src/components/budget/SidebarCategory.tsx (+1 -1)
📝 packages/desktop-client/src/components/budget/SidebarGroup.tsx (+1 -1)
📝 packages/desktop-client/src/components/budget/envelope/CoverMenu.tsx (+1 -1)
📝 packages/desktop-client/src/components/budget/index.tsx (+1 -1)
📝 packages/desktop-client/src/components/budget/tracking/budgetsummary/BudgetTotal.tsx (+19 -10)
📝 packages/desktop-client/src/components/filters/NameFilter.tsx (+1 -1)
📝 packages/desktop-client/src/components/manager/subscribe/Login.tsx (+1 -1)
📝 packages/desktop-client/src/components/manager/subscribe/OpenIdForm.tsx (+2 -2)
📝 packages/desktop-client/src/components/mobile/accounts/Accounts.tsx (+9 -7)
📝 packages/desktop-client/src/components/modals/BudgetPageMenuModal.tsx (+1 -1)
📝 packages/desktop-client/src/components/modals/ConfirmTransactionEditModal.tsx (+33 -26)
📝 packages/desktop-client/src/components/modals/CoverModal.tsx (+1 -1)

...and 25 more files

📄 Description

Carrying on from https://github.com/actualbudget/actual/pull/4041

This is the first of a series of changes to improve the translatable strings generated with react-i18n. At the moment, some sentences are broken up and lose context once exported.

I've also fixed any typos I've seen and migrated every file I've been in so far to use <Trans> inside React components where possible for consistency. There are also some places where the text used for the same action is inconsistent across the app - I've changed these where possible too.

Down to "O" now. I don't think I need to bother with release notes for the rest of these unless you do?


🔄 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/actualbudget/actual/pull/4154 **Author:** [@matt-fidd](https://github.com/matt-fidd) **Created:** 1/14/2025 **Status:** ✅ Merged **Merged:** 1/21/2025 **Merged by:** [@matt-fidd](https://github.com/matt-fidd) **Base:** `master` ← **Head:** `translations-2` --- ### 📝 Commits (10+) - [`7e5c120`](https://github.com/actualbudget/actual/commit/7e5c12054e4392115472144c838cac09f430c0df) translation consistency - [`589b076`](https://github.com/actualbudget/actual/commit/589b0762b0cac627ebdc541929842b5a990c0331) types - [`d329137`](https://github.com/actualbudget/actual/commit/d3291373a25ff8446b5666aba6f248dfbcc5af37) log in vs login - [`412d178`](https://github.com/actualbudget/actual/commit/412d17872bf7b72b11e51a6b9422118d2801ad8d) Update VRT - [`45839e3`](https://github.com/actualbudget/actual/commit/45839e3af0595f3432195351cf8fbaec1d19ed75) Update VRT - [`d3595af`](https://github.com/actualbudget/actual/commit/d3595af4c56c9bec2830191ce8ff8d11c4cb343c) Apply suggestions from code review - [`17beda6`](https://github.com/actualbudget/actual/commit/17beda685ccaef7bff6409a4f62ec0973fd1f194) suggestions - [`58624a4`](https://github.com/actualbudget/actual/commit/58624a44175bac5e865fa96a5e201dcff4cc50d8) note - [`838d126`](https://github.com/actualbudget/actual/commit/838d12607d6ac7707493a73026a8f9791461973a) lint - [`b48df7d`](https://github.com/actualbudget/actual/commit/b48df7dee596b44157c7eca4101264ee708230c4) add context ### 📊 Changes **45 files changed** (+305 additions, -186 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/e2e/settings.mobile.test.js-snapshots/Mobile-Settings-checks-that-settings-page-can-be-opened-1-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/settings.mobile.test.js-snapshots/Mobile-Settings-checks-that-settings-page-can-be-opened-2-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/settings.mobile.test.js-snapshots/Mobile-Settings-checks-that-settings-page-can-be-opened-3-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/src/components/App.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/accounts/Account.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/accounts/Reconcile.tsx` (+10 -5) 📝 `packages/desktop-client/src/components/autocomplete/PayeeAutocomplete.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/budget/BalanceWithCarryover.tsx` (+67 -26) 📝 `packages/desktop-client/src/components/budget/SidebarCategory.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/budget/SidebarGroup.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/budget/envelope/CoverMenu.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/budget/index.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/budget/tracking/budgetsummary/BudgetTotal.tsx` (+19 -10) 📝 `packages/desktop-client/src/components/filters/NameFilter.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/manager/subscribe/Login.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/manager/subscribe/OpenIdForm.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/mobile/accounts/Accounts.tsx` (+9 -7) 📝 `packages/desktop-client/src/components/modals/BudgetPageMenuModal.tsx` (+1 -1) 📝 `packages/desktop-client/src/components/modals/ConfirmTransactionEditModal.tsx` (+33 -26) 📝 `packages/desktop-client/src/components/modals/CoverModal.tsx` (+1 -1) _...and 25 more files_ </details> ### 📄 Description Carrying on from https://github.com/actualbudget/actual/pull/4041 > This is the first of a series of changes to improve the translatable strings generated with react-i18n. At the moment, some sentences are broken up and lose context once exported. > > I've also fixed any typos I've seen and migrated every file I've been in so far to use `<Trans>` inside React components where possible for consistency. There are also some places where the text used for the same action is inconsistent across the app - I've changed these where possible too. Down to "O" now. I don't think I need to bother with release notes for the rest of these unless you do? --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-02-28 21:08:23 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#5219