Skip to content

Conversation

raspberrypitranslationsbot
Copy link
Contributor

No description provided.

@jamiebenstead jamiebenstead temporarily deployed to previews/1212/merge August 28, 2025 15:39 — with GitHub Actions Inactive
@jamiebenstead jamiebenstead temporarily deployed to previews/1212/merge August 28, 2025 16:08 — with GitHub Actions Inactive
@jamiebenstead jamiebenstead temporarily deployed to previews/1212/merge August 29, 2025 09:46 — with GitHub Actions Inactive
@jamiebenstead jamiebenstead temporarily deployed to previews/1212/merge August 29, 2025 09:50 — with GitHub Actions Inactive
@jamiebenstead jamiebenstead temporarily deployed to previews/1212/merge August 29, 2025 10:53 — with GitHub Actions Inactive
@jamiebenstead jamiebenstead temporarily deployed to previews/1212/merge August 29, 2025 11:35 — with GitHub Actions Inactive
@jamiebenstead jamiebenstead temporarily deployed to previews/1212/merge August 29, 2025 11:48 — with GitHub Actions Inactive
@jamiebenstead jamiebenstead temporarily deployed to previews/1212/merge August 29, 2025 11:51 — with GitHub Actions Inactive
@jamiebenstead jamiebenstead temporarily deployed to previews/1212/merge August 29, 2025 12:10 — with GitHub Actions Inactive
@jamiebenstead jamiebenstead temporarily deployed to previews/1212/merge August 29, 2025 12:26 — with GitHub Actions Inactive
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates translations across multiple locales as part of ongoing Crowdin localization efforts. The changes include updates to the i18n configuration, new translation files, and corresponding test updates to use translation keys instead of hardcoded strings.

  • Updated i18n configuration to use a simpler loadPath without environment variable
  • Added new translation files for French (fr-FR), Spanish Latin America (es-LA), and US English (en-US)
  • Updated existing German (xx-XX) and English (en) translations with new keys and content
  • Modified Cypress tests to use translation keys instead of hardcoded English text

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/utils/i18n.js Simplified backend loadPath configuration
public/translations/xx-XX.json Added new German translation keys for UI elements
public/translations/fr-FR.json Complete French translation file
public/translations/es-LA.json Complete Spanish (Latin America) translation file
public/translations/en.json Updated English translations with new keys
public/translations/en-US.json Complete US English translation file
cypress/e2e/*.cy.js Updated test assertions to use translation keys
CHANGELOG.md Added changelog entry for translations

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

@adrian-rpf adrian-rpf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

concerns around removing env

@adrian-rpf adrian-rpf self-requested a review August 29, 2025 13:55
@jamiebenstead jamiebenstead merged commit 238d214 into main Aug 29, 2025
8 checks passed
@jamiebenstead jamiebenstead deleted the l10n_translate-web-component branch August 29, 2025 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants