feat: Italian localization by albanobattistella · Pull Request #33 · panteLx/BetterShift

albanobattistella · 2025-12-06T19:33:39Z

Summary by CodeRabbit

New Features
- Added Italian language support with comprehensive translations for UI strings across many sections (common, validation, forms, app, calendar, color, password, notes, shifts, presets, stats, sync, onboarding, footer, external sync), including labels, placeholders, statuses, errors and action texts with placeholders like {name} and {date}.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Added Italian localization for application messages.

coderabbitai · 2025-12-06T20:03:49Z

Walkthrough

A new Italian localization resource file (messages/it.json) was added, containing translated UI strings across sections such as common, validation, form, app, calendar, password, note, shift, preset, calendar_grid, stats, sync, onboarding, version, footer, and externalSync. No functional logic changes.

Changes

Cohort / File(s)	Summary
Localization Resources `messages/it.json`	New Italian translation file with UI strings for 15+ sections (labels, placeholders, descriptions, statuses, error messages, action texts). Includes interpolation placeholders like `{name}` and `{date}`.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Verify translation accuracy and consistency with Italian conventions
Confirm JSON structure matches other localization files
Ensure interpolation placeholders ({name}, {date}, etc.) align with source keys

Poem

🐰
Saluti in italiano, fresco e snello,
Chiavi e frasi in un gustoso pennello.
Saltello contento tra stringhe e suoni,
Traduzioni pronte, sorrisi e buoni doni.
Viva l'italiano — un piccolo magico anello! 🎉

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title 'feat: Italian localization' clearly and concisely describes the main change: adding Italian language translations to the project.

✨ Finishing touches

📝 Generate docstrings

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment

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

coderabbitai

Actionable comments posted: 2

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b28c649 and dc23ecc.

📒 Files selected for processing (1)

messages/it.json (1 hunks)

🧰 Additional context used

🧠 Learnings (3)

📓 Common learnings

Learnt from: CR
Repo: panteLx/BetterShift PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-12-04T19:12:37.208Z
Learning: Applies to messages/{de,en}.json : All translations for new features must be added to both `messages/de.json` and `messages/en.json`

Learnt from: CR
Repo: panteLx/BetterShift PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-12-04T19:12:37.208Z
Learning: Applies to **/*.tsx : Use `useTranslations()` from next-intl for all user-facing text and load translations from `messages/{de,en}.json`

📚 Learning: 2025-12-04T19:12:37.208Z

Learnt from: CR
Repo: panteLx/BetterShift PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-12-04T19:12:37.208Z
Learning: Applies to messages/{de,en}.json : All translations for new features must be added to both `messages/de.json` and `messages/en.json`

Applied to files:

messages/it.json

📚 Learning: 2025-12-04T19:12:37.208Z

Learnt from: CR
Repo: panteLx/BetterShift PR: 0
File: .github/copilot-instructions.md:0-0
Timestamp: 2025-12-04T19:12:37.208Z
Learning: Applies to **/*.tsx : Use `useTranslations()` from next-intl for all user-facing text and load translations from `messages/{de,en}.json`

Applied to files:

messages/it.json

🪛 Biome (2.1.2)

messages/it.json

[error] 28-28: expected , but instead found {

Remove {

(parse)

[error] 28-28: Property key must be double quoted

(parse)

[error] 28-28: expected : but instead found }

Remove }

(parse)

[error] 28-28: expected , but instead found "? Anche tutti i turni, i preset e le note saranno eliminati."

Remove "? Anche tutti i turni, i preset e le note saranno eliminati."

(parse)

[error] 28-28: expected : but instead found ,

Remove ,

(parse)

[error] 51-51: expected , but instead found {

Remove {

(parse)

[error] 51-51: Property key must be double quoted

(parse)

[error] 51-51: expected : but instead found }

Remove }

(parse)

[error] 51-51: expected , but instead found ""

Remove ""

(parse)

[error] 51-51: expected : but instead found ,

Remove ,

(parse)

[error] 52-52: expected , but instead found {

Remove {

(parse)

[error] 52-52: Property key must be double quoted

(parse)

[error] 52-52: expected : but instead found }

Remove }

(parse)

[error] 52-52: expected , but instead found ""

Remove ""

(parse)

[error] 52-52: expected : but instead found ,

Remove ,

(parse)

[error] 98-98: Expected a property but instead found '"shift_one"'.

Expected a property here.

(parse)

[error] 98-98: Invalid escape sequence

Valid escape sequences are: \\, \/, /", \b\, \f, \n, \r, \t or any unicode escape sequence \uXXXX where X is hexedecimal number.

(parse)

[error] 98-98: End of file expected