Skip to content

feat: added italian language#1048

Merged
lollipopkit merged 2 commits intolollipopkit:mainfrom
loviuz:main
Feb 7, 2026
Merged

feat: added italian language#1048
lollipopkit merged 2 commits intolollipopkit:mainfrom
loviuz:main

Conversation

@loviuz
Copy link
Contributor

@loviuz loviuz commented Feb 6, 2026

Hi, I have added italian translation :-)

Summary by CodeRabbit

  • New Features
    • Added full Italian language support across the application. Users can now choose Italian as their preferred language and see UI labels, buttons, messages, prompts, tips, error notifications, and feature descriptions presented in Italian for a complete localized experience.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 6, 2026

📝 Walkthrough

Walkthrough

This pull request adds a new Italian localization resource file (lib/l10n/app_it.arb). The file sets the locale to "it" and contains ~315 lines of key-value pairs providing Italian translations for UI labels, tips, messages, prompts, and error texts. No functional code changes, API modifications, or exported entity changes are introduced; the change is purely data for localization.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: added italian language' clearly and concisely summarizes the main change—adding Italian localization support to the project.
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

🧹 Recent nitpick comments
lib/l10n/app_it.arb (1)

239-241: Metadata description is in Italian — consider keeping it in English for developer readability.

The @sshHostKeyType description uses Italian ("Etichetta per il tipo di chiave host SSH visualizzata nella finestra di dialogo di verifica della chiave host."). ARB metadata descriptions are developer-facing documentation, not user-facing strings, so keeping them in English would be more consistent for an international contributor base.

Proposed fix
 "@sshHostKeyType": { - "description": "Etichetta per il tipo di chiave host SSH visualizzata nella finestra di dialogo di verifica della chiave host." + "description": "Label for the SSH host key type shown in the host key verification dialog." },
📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 064b677 and d9e457c.

⛔ Files ignored due to path filters (1)
  • lib/generated/l10n/l10n_it.dart is excluded by !**/generated/**
📒 Files selected for processing (1)
  • lib/l10n/app_it.arb
🧰 Additional context used
📓 Path-based instructions (1)
lib/l10n/**/*.arb

📄 CodeRabbit inference engine (CLAUDE.md)

lib/l10n/**/*.arb: After modifying ARB files, run: flutter gen-l10n to regenerate localization
Keep localized strings in ARB files under lib/l10n/ (Flutter i18n)

Files:

  • lib/l10n/app_it.arb
🧠 Learnings (3)
📓 Common learnings
Learnt from: CR Repo: lollipopkit/flutter_server_box PR: 0 File: CLAUDE.md:0-0 Timestamp: 2025-10-08T09:36:41.682Z Learning: Applies to lib/l10n/**/*.arb : Keep localized strings in ARB files under lib/l10n/ (Flutter i18n) 
📚 Learning: 2025-10-08T09:36:41.682Z
Learnt from: CR Repo: lollipopkit/flutter_server_box PR: 0 File: CLAUDE.md:0-0 Timestamp: 2025-10-08T09:36:41.682Z Learning: Applies to lib/l10n/**/*.arb : Keep localized strings in ARB files under lib/l10n/ (Flutter i18n) 

Applied to files:

  • lib/l10n/app_it.arb
📚 Learning: 2025-10-08T09:36:41.682Z
Learnt from: CR Repo: lollipopkit/flutter_server_box PR: 0 File: CLAUDE.md:0-0 Timestamp: 2025-10-08T09:36:41.682Z Learning: Applies to lib/l10n/**/*.arb : After modifying ARB files, run: flutter gen-l10n to regenerate localization 

Applied to files:

  • lib/l10n/app_it.arb
🔇 Additional comments (1)
lib/l10n/app_it.arb (1)

204-204: Previous plural issue has been addressed — "Sensori" is correct.

The translation now correctly uses the Italian plural form "Sensori" for the sensors key.

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.


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.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Fix all issues with AI agents
In `@lib/l10n/app_it.arb`: - Line 204: The translation for the localization key "sensors" is currently singular ("Sensore"); update the value for the "sensors" key to the correct Italian plural "Sensori" so the plurality matches the key (replace the string associated with "sensors" from "Sensore" to "Sensori"). - Line 128: The Italian translation for the localization key "installDockerWithUrl" has awkward word order and is missing the verb; update the value for installDockerWithUrl in lib/l10n/app_it.arb to include the verb "installa" and move the URL to the end of the sentence so it reads naturally in Italian (e.g., "Per favore, installa Docker. Documentazione: https://docs.docker.com/engine/install"); after editing the string run flutter gen-l10n to regenerate the localization artifacts. 
🧹 Nitpick comments (1)
lib/l10n/app_it.arb (1)

239-241: Keep metadata description in English for consistency.

ARB @-prefixed metadata entries (like description) are developer-facing, not user-facing. The description in app_it.arb is in Italian while app_en.arb and other locale files use English for metadata, making it inconsistent.

Proposed fix
 "@sshHostKeyType": { - "description": "Etichetta per il tipo di chiave host SSH visualizzata nella finestra di dialogo di verifica della chiave host." + "description": "Label for the SSH host key type displayed in the host key verification dialog." },
📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a2922f9 and 064b677.

⛔ Files ignored due to path filters (2)
  • lib/generated/l10n/l10n.dart is excluded by !**/generated/**
  • lib/generated/l10n/l10n_it.dart is excluded by !**/generated/**
📒 Files selected for processing (1)
  • lib/l10n/app_it.arb
🧰 Additional context used
📓 Path-based instructions (1)
lib/l10n/**/*.arb

📄 CodeRabbit inference engine (CLAUDE.md)

lib/l10n/**/*.arb: After modifying ARB files, run: flutter gen-l10n to regenerate localization
Keep localized strings in ARB files under lib/l10n/ (Flutter i18n)

Files:

  • lib/l10n/app_it.arb
🧠 Learnings (3)
📓 Common learnings
Learnt from: CR Repo: lollipopkit/flutter_server_box PR: 0 File: CLAUDE.md:0-0 Timestamp: 2025-10-08T09:36:41.682Z Learning: Applies to lib/l10n/**/*.arb : Keep localized strings in ARB files under lib/l10n/ (Flutter i18n) 
📚 Learning: 2025-10-08T09:36:41.682Z
Learnt from: CR Repo: lollipopkit/flutter_server_box PR: 0 File: CLAUDE.md:0-0 Timestamp: 2025-10-08T09:36:41.682Z Learning: Applies to lib/l10n/**/*.arb : Keep localized strings in ARB files under lib/l10n/ (Flutter i18n) 

Applied to files:

  • lib/l10n/app_it.arb
📚 Learning: 2025-10-08T09:36:41.682Z
Learnt from: CR Repo: lollipopkit/flutter_server_box PR: 0 File: CLAUDE.md:0-0 Timestamp: 2025-10-08T09:36:41.682Z Learning: Applies to lib/l10n/**/*.arb : After modifying ARB files, run: flutter gen-l10n to regenerate localization 

Applied to files:

  • lib/l10n/app_it.arb
🔇 Additional comments (2)
lib/l10n/app_it.arb (2)

1-2: Good: File location and locale declaration are correct.

The file is placed under lib/l10n/ as required and the @@locale is properly set to "it". As per coding guidelines, remember to run flutter gen-l10n to regenerate localization after merging this file.


3-315: Overall: Solid Italian translation covering all keys with correct placeholder usage.

The file provides a comprehensive Italian translation with all {placeholder} parameters properly preserved. The translations are generally accurate and natural-sounding. A couple of minor items were flagged above but overall this is good to merge after the small fixes.

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.

@lollipopkit lollipopkit merged commit 89c1d66 into lollipopkit:main Feb 7, 2026
1 check passed
@lollipopkit
Copy link
Owner

:)

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

Labels

None yet

2 participants