Skip to content

feat(lang): add Hungarian language#348

Closed
pisti87 wants to merge 1 commit into
meshtastic:masterfrom
pisti87:patch-4
Closed

feat(lang): add Hungarian language#348
pisti87 wants to merge 1 commit into
meshtastic:masterfrom
pisti87:patch-4

Conversation

@pisti87

@pisti87 pisti87 commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

pls add hungarian lang !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

Summary by CodeRabbit

  • New Features
    • Added Hungarian language support across the application.
    • Translated device status, settings, messaging, operational flows, and storage or telemetry messages.
    • Added localized pluralized counters and dynamic text with preserved formatting.

feat(lang): add Hungarian language
@coderabbitai

coderabbitai Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Adds locale/hu.yml, providing Hungarian translations for application UI, device status, settings, messaging, operational flows, pluralized counters, and storage or telemetry messages.

Changes

Hungarian localization

Layer / File(s) Summary
Hungarian translation catalog
locale/hu.yml
Adds Hungarian translations for user-facing strings, including placeholders, multiline entries, and pluralization blocks.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

Poem

A bunny found new words to share,
Hungarian UI blooms with care.
Status, settings, chats align,
Placeholders hop in every line.
“Nyúl!” says the rabbit, pleased and bright—
The locale now speaks just right.

🚥 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 clearly and concisely describes the main change: adding Hungarian language support.
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.

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
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 3

🤖 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 `@locale/hu.yml`:
- Around line 156-159: Update the Hungarian translations for '%d of %d nodes
online' and 'Filter: %d of %d nodes' to remove the English fragments and use
consistent, fully Hungarian wording in both plural variants, while preserving
the existing format placeholders.
- Line 131: Update the Hungarian translations for both dynamic modem-preset
labels, including the entry at “Modem Preset: %s” and its other variant,
replacing the English label with consistent Hungarian wording while preserving
the %s placeholder.
- Line 4: Replace the untranslated value for the “1 of 1 nodes online” locale
entry with its Hungarian translation, removing the `~` placeholder while
preserving the entry’s existing key and formatting.
🪄 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: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 41201276-2582-4e84-817e-a8581d75797a

📥 Commits

Reviewing files that changed from the base of the PR and between effbb92 and 32d9c73.

📒 Files selected for processing (1)
  • locale/hu.yml

Comment thread locale/hu.yml
hu:
Reboot into BaseUI?: Újraindítás BaseUI módba?
no new messages: nincs új üzenet
1 of 1 nodes online: ~

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

set -euo pipefail

printf '== locale/hu.yml ==\n'
if [ -f locale/hu.yml ]; then
  cat -n locale/hu.yml
else
  echo 'locale/hu.yml not found'
fi

printf '\n== candidate locale files ==\n'
find . -maxdepth 3 -type f \( -path './locale/*' -o -path './locales/*' -o -path './i18n/*' \) | sed 's#^\./##' | sort | head -200

printf '\n== search for locale loading / YAML null handling ==\n'
rg -n --hidden --glob '!**/node_modules/**' --glob '!**/dist/**' --glob '!**/build/**' '\b(locale|locales|i18n|yaml|yml)\b|load.*yaml|parse.*yaml|fallback' .

Repository: meshtastic/device-ui

Length of output: 12861


🏁 Script executed:

set -euo pipefail
cat -n locale/hu.yml 2>/dev/null || true
rg -n --hidden --glob '!**/node_modules/**' --glob '!**/dist/**' --glob '!**/build/**' 'fallback|yaml|yml|locale|locales|i18n|null translation|missing translation' .

Repository: meshtastic/device-ui

Length of output: 25984


🏁 Script executed:

set -euo pipefail
printf 'Files:\n'
find locale -maxdepth 1 -type f | sort
printf '\nhu.yml:\n'
sed -n '1,40p' locale/hu.yml

Repository: meshtastic/device-ui

Length of output: 1988


🏁 Script executed:

set -euo pipefail

printf '== generated Hungarian entry ==\n'
rg -n -A3 -B2 "1 of 1 nodes online|hu_lang|hu_singulars|hu_plurals" locale/lv_i18n.c

printf '\n== README / docs on omitted or null translations ==\n'
sed -n '1,220p' locale/README.md

printf '\n== YAML null handling in locale generator references ==\n'
rg -n "null|omit|missing|fallback|empty string|~" locale -g '!locale/lv_i18n.c' -g '!locale/lv_i18n.h'

Repository: meshtastic/device-ui

Length of output: 32607


🏁 Script executed:

set -euo pipefail

sed -n '3400,3460p' locale/lv_i18n.c
printf '\n---\n'
sed -n '3460,3505p' locale/lv_i18n.c
printf '\n---\n'
sed -n '1,120p' locale/lv_i18n.h

Repository: meshtastic/device-ui

Length of output: 5081


Translate this entry. ~ leaves the source string in place, so this label will still show the English text instead of Hungarian.

🤖 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 `@locale/hu.yml` at line 4, Replace the untranslated value for the “1 of 1
nodes online” locale entry with its Hungarian translation, removing the `~`
placeholder while preserving the entry’s existing key and formatting.

Comment thread locale/hu.yml
'Region: %s': 'Régió: %s'
'User name: %s': 'Felhasználónév: %s'
'Device Role: %s': 'Eszköz szerepe: %s'
'Modem Preset: %s': 'Modem preset: %s'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Translate the dynamic modem-preset labels.

Both variants leave “Modem preset” in English while surrounding labels are Hungarian. Preserve %s, but translate the label consistently.

Also applies to: 140-140

🤖 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 `@locale/hu.yml` at line 131, Update the Hungarian translations for both
dynamic modem-preset labels, including the entry at “Modem Preset: %s” and its
other variant, replacing the English label with consistent Hungarian wording
while preserving the %s placeholder.

Comment thread locale/hu.yml
Comment on lines +156 to +159
'%d of %d nodes online':
one: '%d / %d node online'
other: '%d / %d node online'
'Filter: %d of %d nodes': 'Szűrő: %d / %d node-ok'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Remove English fragments from the node-status translations.

The pluralized and filtered node messages still expose node and online/node-ok, leaving these user-facing strings partially untranslated. Use consistent Hungarian text for both plural variants.

🤖 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 `@locale/hu.yml` around lines 156 - 159, Update the Hungarian translations for
'%d of %d nodes online' and 'Filter: %d of %d nodes' to remove the English
fragments and use consistent, fully Hungarian wording in both plural variants,
while preserving the existing format placeholders.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants