Skip to content

Minor updates and fixes in Czech locale #8122

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 9, 2025

Conversation

latkam
Copy link
Contributor

@latkam latkam commented May 7, 2025

Description

  • fix some typos
  • minor updates of the Czech locale for better language experience
  • translate Electricity load

Preview

Double check

  • I have run pnpx prettier@2 --write . and poetry run format in the top level directory to format my changes.

@latkam latkam force-pushed the update-locale-cs-2 branch from d9922fe to 9955d13 Compare May 7, 2025 20:37
@VIKTORVAV99 VIKTORVAV99 requested review from VIKTORVAV99 and Copilot May 9, 2025 07:59
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR focuses on minor improvements to the Czech locale, including typo fixes and translation updates for energy-related terms.

  • Fixed typos in multiple keys (e.g., "Hodinnový" → "Hodinový").
  • Updated translations to better reflect the intended meanings.
  • Added new translation entries for electricity load and refined existing phrasing.
Comments suppressed due to low confidence (2)

web/src/locales/cs.json:346

  • The key 'load' is very generic compared to the more descriptive 'electricityLoad' used elsewhere. Consider renaming it (e.g., 'networkLoad') for clearer context.
"load": "Zatížení sítě",

web/src/locales/cs.json:322

  • The updated translation for 'utilizing' uses 'využívajíce', which differs from the previous 'využití'. Please review and confirm that this change is consistent with standard usage in the locale.
"utilizing": "využívajíce",

Copy link
Member

@VIKTORVAV99 VIKTORVAV99 left a comment

Choose a reason for hiding this comment

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

LGTM! 🎉

In the future though, please avoid force pushes as it messes with local copies of the git tree (like when I try to review it).

@VIKTORVAV99 VIKTORVAV99 enabled auto-merge (squash) May 9, 2025 08:03
@VIKTORVAV99 VIKTORVAV99 merged commit af11b74 into electricitymaps:master May 9, 2025
23 checks passed
@latkam
Copy link
Contributor Author

latkam commented May 9, 2025

LGTM! 🎉

In the future though, please avoid force pushes as it messes with local copies of the git tree (like when I try to review it).

Ok, sorry, will avoid next time :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants