Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
WalkthroughUpdated Hungarian locale Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@web/src/locales/hu.json`:
- Line 693: Update the translation value for the key
"registration.general.goToToolsRegistrationToLearn" to use the same navigation
separator style as the rest of the locale (replace "->" with ">"), so the string
becomes consistent with other menu path entries in hu.json.
- Line 726: Update the Hungarian translation for the key
"server.actions.extendLicenseForUpdates": replace the awkward phrase "operációs
rendszeri frissítések" with the idiomatic "operációs rendszerfrissítések" so the
value reads "Licenc hosszabbítása az operációs rendszerfrissítések
használatához".
🪄 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: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 2c845af7-726f-4539-acfc-29cf749969ff
📒 Files selected for processing (1)
web/src/locales/hu.json
| "registration.flashProduct": "Flash termék", | ||
| "registration.flashVendor": "Flash gyártó", | ||
| "registration.general.goToToolsRegistrationToLearn": "Menjen a Tartozékok > Regisztráció, hogy többet megtudjon", | ||
| "registration.general.goToToolsRegistrationToLearn": "Lépjen az Eszközök -> Regisztráció oldalra a további információkért", |
There was a problem hiding this comment.
Use consistent navigation separator style.
Line 693 uses ->, while this locale file predominantly uses > for menu paths. Please normalize this string for UI consistency.
💡 Suggested fix
- "registration.general.goToToolsRegistrationToLearn": "Lépjen az Eszközök -> Regisztráció oldalra a további információkért",
+ "registration.general.goToToolsRegistrationToLearn": "Lépjen az Eszközök > Regisztráció oldalra a további információkért",📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| "registration.general.goToToolsRegistrationToLearn": "Lépjen az Eszközök -> Regisztráció oldalra a további információkért", | |
| "registration.general.goToToolsRegistrationToLearn": "Lépjen az Eszközök > Regisztráció oldalra a további információkért", |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@web/src/locales/hu.json` at line 693, Update the translation value for the
key "registration.general.goToToolsRegistrationToLearn" to use the same
navigation separator style as the rest of the locale (replace "->" with ">"), so
the string becomes consistent with other menu path entries in hu.json.
| "server.actions.extendLicenseForUpdates": "Licenc Kiterjesztése az OS Frissítések Engedélyezéséhez", | ||
| "server.actions.extendTrial": "Próbaidő Kiterjesztése", | ||
| "server.actions.disabledTitle": "Ehhez a helyi Unraid API-nak futnia kell", | ||
| "server.actions.extendLicenseForUpdates": "Licenc hosszabbítása az operációs rendszeri frissítések használatához", |
There was a problem hiding this comment.
Polish Hungarian compound wording for OS updates.
Line 726 reads awkwardly with operációs rendszeri frissítések. A more idiomatic form is operációs rendszerfrissítések.
💡 Suggested fix
- "server.actions.extendLicenseForUpdates": "Licenc hosszabbítása az operációs rendszeri frissítések használatához",
+ "server.actions.extendLicenseForUpdates": "Licenc hosszabbítása az operációs rendszerfrissítések használatához",📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| "server.actions.extendLicenseForUpdates": "Licenc hosszabbítása az operációs rendszeri frissítések használatához", | |
| "server.actions.extendLicenseForUpdates": "Licenc hosszabbítása az operációs rendszerfrissítések használatához", |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@web/src/locales/hu.json` at line 726, Update the Hungarian translation for
the key "server.actions.extendLicenseForUpdates": replace the awkward phrase
"operációs rendszeri frissítések" with the idiomatic "operációs
rendszerfrissítések" so the value reads "Licenc hosszabbítása az operációs
rendszerfrissítések használatához".
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1994 +/- ##
==========================================
- Coverage 52.55% 52.55% -0.01%
==========================================
Files 1033 1033
Lines 71649 71649
Branches 8172 8173 +1
==========================================
- Hits 37655 37653 -2
- Misses 33869 33871 +2
Partials 125 125 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@web/src/locales/hu.json`:
- Line 743: Update the localization value for the key
"server.configError.ineligible.action" to use the same menu separator style as
other entries by replacing the arrow "->" with ">" in the string; find the
"server.configError.ineligible.action" entry and change "Eszközök ->
Regisztráció" to "Eszközök > Regisztráció" so it matches the existing locale
conventions.
🪄 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: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 80b64628-e023-459b-9831-0dc88ed9680a
📒 Files selected for processing (1)
web/src/locales/hu.json
| "server.array.warning.started": "Elindítva. Leállítás után a lemeztömb nem fog újraindulni, amíg a fenti probléma nincs megoldva.", | ||
| "server.array.warning.stopped": "Leállítva. A lemeztömb addig nem fog elindulni, amíg a fenti probléma nincs megoldva.", | ||
| "server.cloudError.heading": "Unraid Connect - Hiba", | ||
| "server.configError.ineligible.action": "További információkat az Eszközök -> Regisztráció oldalon talál", |
There was a problem hiding this comment.
Same navigation separator inconsistency.
This line also uses -> instead of > for the menu path separator. For consistency with other parts of the locale file (e.g., lines 406-407), please use >.
💡 Suggested fix
- "server.configError.ineligible.action": "További információkat az Eszközök -> Regisztráció oldalon talál",
+ "server.configError.ineligible.action": "További információkat az Eszközök > Regisztráció oldalon talál",📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| "server.configError.ineligible.action": "További információkat az Eszközök -> Regisztráció oldalon talál", | |
| "server.configError.ineligible.action": "További információkat az Eszközök > Regisztráció oldalon talál", |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@web/src/locales/hu.json` at line 743, Update the localization value for the
key "server.configError.ineligible.action" to use the same menu separator style
as other entries by replacing the arrow "->" with ">" in the string; find the
"server.configError.ineligible.action" entry and change "Eszközök ->
Regisztráció" to "Eszközök > Regisztráció" so it matches the existing locale
conventions.
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (2)
web/src/locales/hu.json (2)
693-693:⚠️ Potential issue | 🟡 MinorNormalize menu path separators to
>for consistency.Line 693, Line 743, Line 755, Line 780, and Line 823 still use
->, while most navigation-path strings use>.💡 Suggested fix
- "registration.general.goToToolsRegistrationToLearn": "Lépjen az Eszközök -> Regisztráció oldalra a további információkért", + "registration.general.goToToolsRegistrationToLearn": "Lépjen az Eszközök > Regisztráció oldalra a további információkért", - "server.configError.ineligible.action": "További információkat az Eszközök -> Regisztráció oldalon talál", + "server.configError.ineligible.action": "További információkat az Eszközök > Regisztráció oldalon talál", - "server.deprecatedSsl.message": "Az unraid.net SSL-tanúsítványai 2023. január 1-jén lejártak. Az új myunraid.net-tartományunk használatához gondoskodnia KELL egy új SSL-tanúsítványról. Ezt megteheti a Beállítások -> Rendszergazdai hozzáférés oldalon.", + "server.deprecatedSsl.message": "Az unraid.net SSL-tanúsítványai 2023. január 1-jén lejártak. Az új myunraid.net-tartományunk használatához gondoskodnia KELL egy új SSL-tanúsítványról. Ezt megteheti a Beállítások > Rendszergazdai hozzáférés oldalon.", - "server.state.enoconn.message": "<p>A próbalicenc kulcsához internetkapcsolat szükséges. Kérjük, ellenőrizze a </p><p><a href=\"/Settings/NetworkSettings\" class=\"underline\">Beállítások -> Hálózat</a> oldalt.</p>", + "server.state.enoconn.message": "<p>A próbalicenc kulcsához internetkapcsolat szükséges. Kérjük, ellenőrizze a </p><p><a href=\"/Settings/NetworkSettings\" class=\"underline\">Beállítások > Hálózat</a> oldalt.</p>", - "updateOs.checkUpdateResponseModal.goToSettingsNotificationsToEnable": "Lépjen a Beállítások -> Értesítések oldalra az operációs rendszer automatikus frissítési értesítéseinek engedélyezéséhez a későbbi kiadásoknál.", + "updateOs.checkUpdateResponseModal.goToSettingsNotificationsToEnable": "Lépjen a Beállítások > Értesítések oldalra az operációs rendszer automatikus frissítési értesítéseinek engedélyezéséhez a későbbi kiadásoknál.",Also applies to: 743-743, 755-755, 780-780, 823-823
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@web/src/locales/hu.json` at line 693, Several navigation-path translations use the "->" separator instead of the project's normalized ">"; update the affected locale keys (e.g., registration.general.goToToolsRegistrationToLearn and the other similar keys noted in the comment) to replace "->" with ">" so all navigation-path strings use the same separator. Locate the strings in web/src/locales/hu.json by the keys shown and change "Eszközök -> Regisztráció" (and the other occurrences) to use "Eszközök > Regisztráció" (preserve surrounding text and punctuation).
726-726:⚠️ Potential issue | 🟡 MinorUse the idiomatic compound form for OS updates.
Line 726 reads awkwardly as
operációs rendszeri frissítések; preferoperációs rendszerfrissítések.💡 Suggested fix
- "server.actions.extendLicenseForUpdates": "Licenc hosszabbítása az operációs rendszeri frissítések használatához", + "server.actions.extendLicenseForUpdates": "Licenc hosszabbítása az operációs rendszerfrissítések használatához",🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@web/src/locales/hu.json` at line 726, Update the Hungarian translation for the key "server.actions.extendLicenseForUpdates": replace the phrase "operációs rendszeri frissítések" with the idiomatic compound "operációs rendszerfrissítések" so the value becomes "Licenc hosszabbítása az operációs rendszerfrissítések használatához" to fix the awkward wording.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@web/src/locales/hu.json`:
- Line 849: In the translation value for the key
"updateOs.pleaseFinishTheInitiatedDowngradeTo" replace the incorrect article "az
megkezdett" with "a megkezdett" so the string reads correctly as "Kérjük,
fejezze be a megkezdett korábbi verzióra történő visszaállást a frissítések
engedélyezéséhez."; locate the key
"updateOs.pleaseFinishTheInitiatedDowngradeTo" and update its value accordingly.
---
Duplicate comments:
In `@web/src/locales/hu.json`:
- Line 693: Several navigation-path translations use the "->" separator instead
of the project's normalized ">"; update the affected locale keys (e.g.,
registration.general.goToToolsRegistrationToLearn and the other similar keys
noted in the comment) to replace "->" with ">" so all navigation-path strings
use the same separator. Locate the strings in web/src/locales/hu.json by the
keys shown and change "Eszközök -> Regisztráció" (and the other occurrences) to
use "Eszközök > Regisztráció" (preserve surrounding text and punctuation).
- Line 726: Update the Hungarian translation for the key
"server.actions.extendLicenseForUpdates": replace the phrase "operációs
rendszeri frissítések" with the idiomatic compound "operációs
rendszerfrissítések" so the value becomes "Licenc hosszabbítása az operációs
rendszerfrissítések használatához" to fix the awkward wording.
🪄 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: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 10164c51-54b6-4ec0-a837-596f242469ba
📒 Files selected for processing (1)
web/src/locales/hu.json
| "updateOs.ineligible.osVersionRequired": "Az operációs rendszeri frissítések ellenőrzéséhez az operációs rendszer érvényes verziója szükséges.", | ||
| "updateOs.ineligible.updatesExpired": "Az Ön {0} licence egy év ingyenes frissítést tartalmazott a vásárláskor. Most meghosszabbíthatja licencét és hozzáférhet a legújabb operációs rendszeri frissítésekhez.", | ||
| "updateOs.ineligible.updatesExpiredWithAvailable": "Az Ön {0} licence egy év ingyenes frissítést tartalmazott a vásárláskor. Most jogosult a licence meghosszabbítására és a legújabb operációs rendszeri frissítésekre. Továbbra is jogosult a(z) {1}-i vagy azelőtt közzétett operációs rendszeri frissítések használatára.", | ||
| "updateOs.pleaseFinishTheInitiatedDowngradeTo": "Kérjük, fejezze be az megkezdett korábbi verzióra történő visszaállást a frissítések engedélyezéséhez.", |
There was a problem hiding this comment.
Fix article typo in downgrade prompt.
Line 849 uses az megkezdett; this should be a megkezdett.
💡 Suggested fix
- "updateOs.pleaseFinishTheInitiatedDowngradeTo": "Kérjük, fejezze be az megkezdett korábbi verzióra történő visszaállást a frissítések engedélyezéséhez.",
+ "updateOs.pleaseFinishTheInitiatedDowngradeTo": "Kérjük, fejezze be a megkezdett korábbi verzióra történő visszaállást a frissítések engedélyezéséhez.",📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| "updateOs.pleaseFinishTheInitiatedDowngradeTo": "Kérjük, fejezze be az megkezdett korábbi verzióra történő visszaállást a frissítések engedélyezéséhez.", | |
| "updateOs.pleaseFinishTheInitiatedDowngradeTo": "Kérjük, fejezze be a megkezdett korábbi verzióra történő visszaállást a frissítések engedélyezéséhez.", |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@web/src/locales/hu.json` at line 849, In the translation value for the key
"updateOs.pleaseFinishTheInitiatedDowngradeTo" replace the incorrect article "az
megkezdett" with "a megkezdett" so the string reads correctly as "Kérjük,
fejezze be a megkezdett korábbi verzióra történő visszaállást a frissítések
engedélyezéséhez."; locate the key
"updateOs.pleaseFinishTheInitiatedDowngradeTo" and update its value accordingly.
|
This plugin has been deployed to Cloudflare R2 and is available for testing. |
1 participant
Summary by CodeRabbit