Skip to content

fix(i18n-lv): fill in missing and re-review previous Latvian localization - #45844

Open
EPoikans wants to merge 4 commits into
apache:masterfrom
EPoikans:fix/i18n-lv
Open

EPoikans wants to merge 4 commits into
apache:masterfrom
EPoikans:fix/i18n-lv

Conversation

@EPoikans

Copy link
Copy Markdown

SUMMARY

Completes the Latvian (lv) catalog and makes it consistent.

  • Translates the 1,095 untranslated entries and reviews the 375 fuzzy ones: 0 untranslated and 0 fuzzy remain.
  • Uses one Latvian term per concept across the catalog (e.g. dashboard = infopanelis, report = atskaite). The formal Jūs is always capitalised.
  • Checks short strings and fragments against the code that uses them (button vs. label, agreement, case where strings are joined).
  • Removes the backfill Machine-translated comments and fuzzy flags from the reviewed entries.

Drafted and cross-reviewed with Claude models. A native Latvian speaker made the terminology decisions and settled the disputed entries.

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

@github-actions github-actions Bot added i18n Namespace | Anything related to localization i18n:latvian labels Oct 29, 2026
@codecov

codecov Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.96%. Comparing base (664e09d) to head (7007874).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #45844       +/-   ##
===========================================
+ Coverage   65.35%   84.96%   +19.61%     
===========================================
  Files        3083     3074        -9     
  Lines      211333   209716     -1617     
  Branches    49823    49058      -765     
===========================================
+ Hits       138108   178177    +40069     
+ Misses      70043    28137    -41906     
- Partials     3182     3402      +220     
Flag Coverage Δ
hive 34.05% <ø> (ø)
mysql 51.81% <ø> (ø)
postgres 51.84% <ø> (ø)
presto 35.69% <ø> (ø)
python 88.11% <ø> (+35.98%) ⬆️
sqlite 51.52% <ø> (ø)
unit 83.23% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@EPoikans
EPoikans marked this pull request as ready for review October 29, 2026 12:33
@codeant-ai-for-open-source

codeant-ai-for-open-source Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

CodeAnt  CodeAnt PR Risk

🟢  Low Risk

  • The PR appears safe to merge; no actionable issues were identified in the review.

Assessed commit: 7007874bd989

@sadpandajoe
sadpandajoe requested a review from rusackas October 29, 2026 18:31
@github-actions github-actions Bot added the requires:rebase Requires rebasing on top of current master label Oct 31, 2026
@github-actions github-actions Bot removed the requires:rebase Requires rebasing on top of current master label Nov 11, 2026

This branch has not been deployed

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

Labels

i18n:latvian i18n Namespace | Anything related to localization size/XXL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant