[No QA] Translate 'Expensify Card' across all locales#83006
[No QA] Translate 'Expensify Card' across all locales#83006roryabraham merged 4 commits intomainfrom
Conversation
Add glossary entries mapping 'Expensify Card' to locale-appropriate translations (e.g. Tarjeta Expensify, Carte Expensify, Expensify Karte) and retranslate all affected phrases via generateTranslations.ts. Co-authored-by: Cursor <cursoragent@cursor.com>
Translate 'Expensify Card Liability Account' in de.ts and nl.ts, and the WITHDRAWAL_TYPE.EXPENSIFY_CARD label in de.ts, nl.ts, es.ts. Co-authored-by: Cursor <cursoragent@cursor.com>
|
@heyjennahay not sure we need a product review here. Just changing translation strings. Maybe you could buddy check Dutch if you wanted 🙂 |
|
@eh2077 no C+ review needed |
| updatedFeatureEnabled: ({enabled, featureName}: {enabled: boolean; featureName: string}) => { | ||
| switch (featureName) { | ||
| case 'categories': | ||
| return `${enabled ? 'Activé' : 'Désactivé'} catégories`; |
There was a problem hiding this comment.
Most of these sound really odd
There was a problem hiding this comment.
(But NAB as it's unrelated to the Expensify Card changes)
There was a problem hiding this comment.
lol, that's not reassuring given that French is the one locale we already audited
Reviewer Checklist
Screenshots/VideosAndroid: HybridAppAndroid: mWeb ChromeiOS: HybridAppiOS: mWeb SafariMacOS: Chrome / Safari |
|
🚧 @roryabraham has triggered a test Expensify/App build. You can view the workflow run here. |
|
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
|
🧪🧪 Use the links below to test this adhoc build on Android, iOS, and Web. Happy testing! 🧪🧪
|
|
🚀 Deployed to staging by https://github.com/roryabraham in version: 9.3.24-1 🚀
|
|
🚀 Deployed to production by https://github.com/puneetlath in version: 9.3.24-3 🚀
|
Explanation of Change
generateTranslations.ts --paths. For example:Fixed Issues
$ #64478
Tests
n/a - just updating UI strings.
Offline tests
N/A - translation-only change.
QA Steps
N/A - translation-only change.
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectioncanBeMissingparam foruseOnyxtoggleReportand notonIconClick)src/languages/*files and using the translation methodSTYLE.md) were followedAvatar, I verified the components usingAvatarare working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))npm run compress-svg)Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.ScrollViewcomponent to make it scrollable when more elements are added to the page.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari