Skip to content

Update Italian translations#1787

Open
albanobattistella wants to merge 1 commit into
saber-notes:mainfrom
albanobattistella:patch-76
Open

Update Italian translations#1787
albanobattistella wants to merge 1 commit into
saber-notes:mainfrom
albanobattistella:patch-76

Conversation

@albanobattistella

@albanobattistella albanobattistella commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Summary by cubic

Updated Italian translations for home sorting and layout labels to replace outdated English strings and improve localization in the UI. Replaces "Sort by", "Name (A-Z)", and layout options with "Ordina per", "Nome (A-Z)", and Italian grid labels.

Note: Saber is trialing Cubic AI code review. AI output may contain mistakes, misconceptions, and hallucinations. You can act on the AI feedback if you find it helpful and are sure it's correct; otherwise you can disregard it and wait for a human reviewer.

Written for commit 1a1876a. Summary will update on new commits.

@codecov

codecov Bot commented Jul 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 51.76%. Comparing base (769e8d7) to head (1a1876a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1787   +/-   ##
=======================================
  Coverage   51.76%   51.76%           
=======================================
  Files         133      133           
  Lines        9467     9467           
=======================================
  Hits         4901     4901           
  Misses       4566     4566           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@cubic-dev-ai cubic-dev-ai 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.

2 issues found across 1 file

Confidence score: 3/5

  • In lib/i18n/_missing_translations.yaml, the new Italian sort/layout text appears to be updated only in the missing-translations tracker, so the app will likely keep showing the old strings after merge; this makes the PR’s user-facing i18n change ineffective — apply the same entries to the actual locale source used for translation generation before merging.
  • lib/i18n/_missing_translations.yaml is missing the required AI-agent credit header, which can cause repo policy/check failures even though runtime risk is low — add the AGENTS.md-required credit comment at the top before merging.
Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="lib/i18n/_missing_translations.yaml">

<violation number="1" location="lib/i18n/_missing_translations.yaml:627">
P3: The changed translation file is missing the required AI-agent credit header. Adding the prescribed credit comment at the top would satisfy the repository rule in `AGENTS.md`.</violation>

<violation number="2" location="lib/i18n/_missing_translations.yaml:629">
P2: The Italian UI will continue showing the old sort/layout strings because these edits only update `_missing_translations.yaml`, not the locale source consumed to generate the app translations. Applying the entries to `it.i18n.yaml` through the documented workflow is needed for the new text to appear.

(Based on your team's feedback about editing translation sources instead of generated/intermediate files.) [FEEDBACK_USED]</violation>
</file>

Tip: cubic used a learning from your PR history. Let your coding agent read cubic learnings directly with the cubic MCP.

Re-trigger cubic

lastModifiedOldToNew(OUTDATED): Edited (Oldest first)
sortBy(OUTDATED): Ordina per
nameAToZ(OUTDATED): Nome (A-Z)
nameZToA(OUTDATED): Nome (Z-A)

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.

P2: The Italian UI will continue showing the old sort/layout strings because these edits only update _missing_translations.yaml, not the locale source consumed to generate the app translations. Applying the entries to it.i18n.yaml through the documented workflow is needed for the new text to appear.

(Based on your team's feedback about editing translation sources instead of generated/intermediate files.)

View Feedback

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At lib/i18n/_missing_translations.yaml, line 629:

<comment>The Italian UI will continue showing the old sort/layout strings because these edits only update `_missing_translations.yaml`, not the locale source consumed to generate the app translations. Applying the entries to `it.i18n.yaml` through the documented workflow is needed for the new text to appear.

(Based on your team's feedback about editing translation sources instead of generated/intermediate files.) </comment>

<file context>
@@ -624,15 +624,15 @@ hu:
-      lastModifiedOldToNew(OUTDATED): Edited (Oldest first)
+      sortBy(OUTDATED): Ordina per
+      nameAToZ(OUTDATED): Nome (A-Z)
+      nameZToA(OUTDATED): Nome (Z-A)
+      lastModifiedNewToOld(OUTDATED): Modificato (Dal più recente)
+      lastModifiedOldToNew(OUTDATED): Modificato (Dal più vecchio)
</file context>

@@ -624,15 +624,15 @@ hu:
it:

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.

P3: The changed translation file is missing the required AI-agent credit header. Adding the prescribed credit comment at the top would satisfy the repository rule in AGENTS.md.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At lib/i18n/_missing_translations.yaml, line 627:

<comment>The changed translation file is missing the required AI-agent credit header. Adding the prescribed credit comment at the top would satisfy the repository rule in `AGENTS.md`.</comment>

<file context>
@@ -624,15 +624,15 @@ hu:
-      nameZToA(OUTDATED): Name (Z-A)
-      lastModifiedNewToOld(OUTDATED): Edited (Newest first)
-      lastModifiedOldToNew(OUTDATED): Edited (Oldest first)
+      sortBy(OUTDATED): Ordina per
+      nameAToZ(OUTDATED): Nome (A-Z)
+      nameZToA(OUTDATED): Nome (Z-A)
</file context>

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.

1 participant