Skip to content

Filled up empty translation strings in German Spanish French and Dutch#316

Closed
MsVivienne wants to merge 1 commit intoDembrane:mainfrom
MsVivienne:Filled-up-empty-translation-msgstrings
Closed

Filled up empty translation strings in German Spanish French and Dutch#316
MsVivienne wants to merge 1 commit intoDembrane:mainfrom
MsVivienne:Filled-up-empty-translation-msgstrings

Conversation

@MsVivienne
Copy link
Copy Markdown
Contributor

@MsVivienne MsVivienne commented Oct 1, 2025

Summary by CodeRabbit

  • New Features
    • Expanded localization coverage: completed and updated German, Spanish, French, and Dutch translations for playback controls, transcript status, error messages, and navigation labels. Examples include “Conversation Audio,” “Follow/Follow playback,” “No content,” “Now,” and “Show timestamps (experimental).” Users should see more consistent, fully localized UI text across these areas.
  • Chores
    • Minor locale formatting adjustments with no functional impact.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Oct 1, 2025

Walkthrough

Locale updates across de-DE, es-ES, fr-FR, and nl-NL PO files to fill previously empty translations; minor non-functional tweak in es-ES.ts. No logic, exports, or control-flow changes—only string translations and formatting.

Changes

Cohort / File(s) Summary of changes
German locale updates
echo/frontend/src/locales/de-DE.po
Filled missing German translations for navigation, status, timestamps, and transcription-related strings; display-only text changes.
Spanish locale updates
echo/frontend/src/locales/es-ES.po, echo/frontend/src/locales/es-ES.ts
Populated Spanish translations for UI/status/transcription strings; .ts retains same export structure with minor formatting/no-op change.
French locale updates
echo/frontend/src/locales/fr-FR.po
Added French translations across numerous UI strings, including transcription, playback, and error messages; some deprecated entries remain marked.
Dutch locale updates
echo/frontend/src/locales/nl-NL.po
Completed Dutch translations for labels, statuses, timestamps, and transcription messages; display-only updates.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

improvement

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed This title directly and succinctly summarizes the primary change by indicating that empty translation strings were filled for German, Spanish, French, and Dutch, which matches the modifications in the PR without extraneous detail.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 387119c and 691a900.

📒 Files selected for processing (5)
  • echo/frontend/src/locales/de-DE.po (13 hunks)
  • echo/frontend/src/locales/es-ES.po (13 hunks)
  • echo/frontend/src/locales/es-ES.ts (1 hunks)
  • echo/frontend/src/locales/fr-FR.po (13 hunks)
  • echo/frontend/src/locales/nl-NL.po (13 hunks)
🧰 Additional context used
🧬 Code graph analysis (1)
echo/frontend/src/locales/es-ES.ts (1)
echo/frontend/src/locales/en-US.ts (1)
  • messages (1-1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: ci-check-server

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants