Skip to content

Additional SE translations for new features#148

Open
ulfhedlund wants to merge 2 commits into
EdgeTX:mainfrom
ulfhedlund:SE-translations
Open

Additional SE translations for new features#148
ulfhedlund wants to merge 2 commits into
EdgeTX:mainfrom
ulfhedlund:SE-translations

Conversation

@ulfhedlund

@ulfhedlund ulfhedlund commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Documentation
    • Expanded Swedish localization with updated translated strings for backup/restore operations
    • Updated Swedish UI labels, status messages, and error notifications for device flashing/bootloader workflows
    • Improved Swedish wording across prompts and workflow guidance, including conflict-handling and messaging text

@coderabbitai

coderabbitai Bot commented Jun 16, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 14a04e65-0343-4bb4-baec-6cd16aa04c14

📥 Commits

Reviewing files that changed from the base of the PR and between 49d8a32 and 79918f1.

📒 Files selected for processing (1)
  • locales/sv/backup.json
✅ Files skipped from review due to trivial changes (1)
  • locales/sv/backup.json

📝 Walkthrough

Walkthrough

Swedish locale files backup.json and flashing.json are updated to replace English placeholder strings with Swedish translations. Coverage spans backup/restore UI, model conflict prompts, bootloader/firmware flashing statuses, error messages, and workflow step labels. One typo is present in backup.json.

Changes

Swedish Locale Translation Updates

Layer / File(s) Summary
Backup/restore strings
locales/sv/backup.json
Replaces English placeholder values with Swedish across backup creation, restore flow, conflict handling, and UI labels.
Flashing workflow strings
locales/sv/flashing.json
Replaces English placeholders with Swedish for bootloader erase/flash statuses, cloudbuild download, "Could not …" error messages, download/erase/flash progress labels, overview & flash guidance, reboot/remove flow steps, and write bootloader/firmware labels.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title refers to adding Swedish translations for new features, which aligns with the changeset's core purpose of updating Swedish locale strings in backup and flashing JSON files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@locales/sv/backup.json`:
- Line 48: The Swedish translation for the "Overwrite all" key at line 48
contains a typo: the word "alltl" should be corrected to "allt". Locate the
"Overwrite all" entry in the backup.json file and fix the translation value by
removing the extra "l" at the end, changing "Skriv över alltl" to "Skriv över
allt", which matches the correct usage found elsewhere in the file.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: af13a9b3-3e9e-477a-a0c1-4d8aedb45b8b

📥 Commits

Reviewing files that changed from the base of the PR and between eb9b4d7 and 49d8a32.

📒 Files selected for processing (2)
  • locales/sv/backup.json
  • locales/sv/flashing.json

Comment thread locales/sv/backup.json Outdated
@codecov

codecov Bot commented Jun 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 34.95%. Comparing base (eb9b4d7) to head (79918f1).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #148   +/-   ##
=======================================
  Coverage   34.95%   34.95%           
=======================================
  Files         115      115           
  Lines        4288     4288           
  Branches     1033     1033           
=======================================
  Hits         1499     1499           
  Misses       2585     2585           
  Partials      204      204           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

1 participant