Skip to content

New Crowdin updates#2156

Open
myieye wants to merge 1 commit intodevelopfrom
l10n_develop
Open

New Crowdin updates#2156
myieye wants to merge 1 commit intodevelopfrom
l10n_develop

Conversation

@myieye
Copy link
Collaborator

@myieye myieye commented Feb 5, 2026

No description provided.

@github-actions github-actions bot added the 💻 FW Lite issues related to the fw lite application, not miniLcm or crdt related label Feb 5, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 5, 2026

📝 Walkthrough

Walkthrough

This pull request removes two translation entries from the English locale file (en.po): a "Filters" header entry associated with SearchFilter.svelte and a "Preview not available" entry from ActivityItemChangePreview.svelte. No functional logic or control flow modifications are made.

Changes

Cohort / File(s) Summary
Localization Updates
frontend/viewer/src/locales/en.po
Removed two translation string entries: "Filters" header and "Preview not available" message.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • New Crowdin updates #2130: Also modifies frontend/viewer/src/locales/en.po with overlapping translation string removals for SearchFilter and activity preview text.

Suggested labels

💻 FW Lite

Poem

🐰 Two strings hop away today,
"Filters" and "Preview" bid adieu,
The locale file so light and clean,
Less translation clutter—hooray!
Simplicity hops through the code. ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (2 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'New Crowdin updates' is generic and vague, using a non-descriptive term that doesn't clearly convey what was changed in the translation files. Consider a more specific title like 'Remove translation entries from en.po' or 'Update en.po translations' to better describe the actual changes made.
Description check ❓ Inconclusive No pull request description was provided by the author, making it impossible to assess the intent or context of the translation changes. Add a description explaining why the translation entries were removed and what the Crowdin update entails for better context.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • 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 and usage tips.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

UI unit Tests

  1 files  ±0   50 suites  ±0   25s ⏱️ -1s
138 tests ±0  138 ✅ ±0  0 💤 ±0  0 ❌ ±0 
203 runs  ±0  203 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 3290581. ± Comparison against base commit f37f947.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

C# Unit Tests

146 tests  ±0   146 ✅ ±0   19s ⏱️ ±0s
 22 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 3290581. ± Comparison against base commit f37f947.

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