Skip to content

New Crowdin updates#2181

Open
myieye wants to merge 8 commits intodevelopfrom
l10n_develop
Open

New Crowdin updates#2181
myieye wants to merge 8 commits intodevelopfrom
l10n_develop

Conversation

@myieye
Copy link
Collaborator

@myieye myieye commented Feb 24, 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 24, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d3a5b3e and 97c20f5.

📒 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

📝 Walkthrough

Walkthrough

This PR adds new validation message translations across eight locale files (English, Spanish, French, Indonesian, Korean, Malay, Swahili, and Vietnamese) for the NewEntryDialog component, introducing required-field validation messages for definition, lexeme form, and word inputs.

Changes

Cohort / File(s) Summary
Internationalization - NewEntryDialog Validations
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
Added three new validation message translations: "Definition or Gloss is required", "Lexeme form or Citation form is required", and "Word or Display as is required" across all locales. Updated PO-Revision-Date metadata.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Suggested labels

💻 FW Lite

Suggested reviewers

  • hahn-kev
  • imnasnainaec

Poem

🐰 A rabbit hops through translations so fine,
Adding validation messages in every line,
From Spanish to Swahili, Indonesian to French,
Eight locales blessed, validation's on the bench!
Required fields now whisper across the globe,
In every tongue, users' paths are strobed. ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No description was provided by the author, which is a significant omission for a pull request that updates translation files across 8 different locales. Add a description explaining the purpose of these translation updates and why these specific validation messages were added to the NewEntryDialog component.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'New Crowdin updates' accurately describes the main change—adding new internationalization entries across multiple locale files (en.po, es.po, fr.po, id.po, ko.po, ms.po, sw.po, vi.po) from Crowdin translations.
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.

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