Skip to content

New Crowdin updates#2355

Open
myieye wants to merge 57 commits into
developfrom
l10n_develop
Open

New Crowdin updates#2355
myieye wants to merge 57 commits into
developfrom
l10n_develop

Conversation

@myieye

@myieye myieye commented Jun 13, 2026

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • New Features
    • Added and expanded translated UI text for activity filters, sorting, loading/empty states, project creation, offline/login flows, and error recovery.
  • Bug Fixes
    • Added clearer messages for view failures, sync issues, task errors, and project creation failures.
  • Documentation
    • Updated locale metadata and source references across supported languages.
  • Chores
    • Removed an obsolete placeholder string and refreshed translation catalog formatting.

@github-actions github-actions Bot added the 💻 FW Lite issues related to the fw lite application, not miniLcm or crdt related label Jun 13, 2026
@coderabbitai

coderabbitai Bot commented Jun 13, 2026

Copy link
Copy Markdown

Review Change Stack

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 3226fecf-a71d-4e21-a6bb-afd65610fc60

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

This PR updates locale catalogs for activity filtering and status text, project creation and offline/sync flows, and shared browse/view/task error messages across English and multiple translated languages.

Changes

Locale string updates

Layer / File(s) Summary
Activity filter and status strings
frontend/viewer/src/locales/en.po, frontend/viewer/src/locales/es.po, frontend/viewer/src/locales/fr.po, frontend/viewer/src/locales/id.po, frontend/viewer/src/locales/ko.po, frontend/viewer/src/locales/ms.po, frontend/viewer/src/locales/sw.po, frontend/viewer/src/locales/vi.po
Adds activity filter labels, loading/empty states, sort options, and related source-reference updates across the locale catalogs.
Project creation and offline strings
frontend/viewer/src/locales/en.po, frontend/viewer/src/locales/es.po, frontend/viewer/src/locales/fr.po, frontend/viewer/src/locales/id.po, frontend/viewer/src/locales/ko.po, frontend/viewer/src/locales/ms.po, frontend/viewer/src/locales/sw.po, frontend/viewer/src/locales/vi.po
Adds project creation labels, vernacular-writing-system strings, offline/login/sync messages, and placeholder removals or context updates.
View and task error messages
frontend/viewer/src/locales/en.po, frontend/viewer/src/locales/es.po, frontend/viewer/src/locales/fr.po, frontend/viewer/src/locales/id.po, frontend/viewer/src/locales/ko.po, frontend/viewer/src/locales/ms.po, frontend/viewer/src/locales/sw.po, frontend/viewer/src/locales/vi.po
Adds browse, generic view, task, and retry error strings, plus supporting ViewErrorBoundary and fallback source-reference adjustments.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

Suggested reviewers: rmunn

🚥 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 matches the PR’s main change: localized translation updates synced from Crowdin.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch l10n_develop

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.

@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 `@frontend/viewer/src/locales/ms.po`:
- Around line 1744-1747: The Malay translation for the msgid "Something went
wrong" in the gettext entry is awkward; update the msgstr for that msgid (the
entry with msgid "Something went wrong" used by ViewErrorBoundary) to a more
natural Malay phrase such as "Maaf, sesuatu telah berlaku." so the generic error
message reads idiomatically.
🪄 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: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 97fe7eeb-e593-4cc4-8ada-d90bf07ca304

📥 Commits

Reviewing files that changed from the base of the PR and between 248ea03 and 28ddd53.

📒 Files selected for processing (8)
  • frontend/viewer/src/locales/en.po
  • frontend/viewer/src/locales/es.po
  • frontend/viewer/src/locales/fr.po
  • frontend/viewer/src/locales/id.po
  • frontend/viewer/src/locales/ko.po
  • frontend/viewer/src/locales/ms.po
  • frontend/viewer/src/locales/sw.po
  • frontend/viewer/src/locales/vi.po

Comment on lines +1744 to +1747
#: src/lib/layout/ViewErrorBoundary.svelte
msgid "Something went wrong"
msgstr "Sesuatu telah berlaku dengan salah"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Refine Malay phrasing for the generic error message.

The current msgstr for “Something went wrong” reads unnaturally in Malay and can confuse users. A more idiomatic phrasing would improve clarity.

Suggested wording update
 msgid "Something went wrong"
-msgstr "Sesuatu telah berlaku dengan salah"
+msgstr "Sesuatu tidak kena"
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
#: src/lib/layout/ViewErrorBoundary.svelte
msgid "Something went wrong"
msgstr "Sesuatu telah berlaku dengan salah"
#: src/lib/layout/ViewErrorBoundary.svelte
msgid "Something went wrong"
msgstr "Sesuatu tidak kena"
🤖 Prompt for 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.

In `@frontend/viewer/src/locales/ms.po` around lines 1744 - 1747, The Malay
translation for the msgid "Something went wrong" in the gettext entry is
awkward; update the msgstr for that msgid (the entry with msgid "Something went
wrong" used by ViewErrorBoundary) to a more natural Malay phrase such as "Maaf,
sesuatu telah berlaku." so the generic error message reads idiomatically.

myieye added 30 commits June 26, 2026 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💻 FW Lite issues related to the fw lite application, not miniLcm or crdt related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant