Skip to content

Add LLM translations for French and Spanish - #811

Merged
pmarkowsky merged 1 commit into
northpolesec:mainfrom
pmarkowsky:plm/add-french-and-spanish-with-claude
Feb 24, 2026
Merged

Add LLM translations for French and Spanish#811
pmarkowsky merged 1 commit into
northpolesec:mainfrom
pmarkowsky:plm/add-french-and-spanish-with-claude

Conversation

@pmarkowsky

Copy link
Copy Markdown
Member

I used Claude Code to add internationalization for French, Canadian French, and Spanish.

@github-actions github-actions Bot added comp/gui Issues or PRs related to the Santa GUI size/l Size: large labels Feb 24, 2026
@pmarkowsky
pmarkowsky marked this pull request as ready for review February 24, 2026 20:29
@pmarkowsky
pmarkowsky requested a review from a team as a code owner February 24, 2026 20:29
@coderabbitai

coderabbitai Bot commented Feb 24, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between ab1e421 and d7d9a89.

📒 Files selected for processing (3)
  • Source/gui/Resources/es.lproj/Localizable.strings
  • Source/gui/Resources/fr-CA.lproj/Localizable.strings
  • Source/gui/Resources/fr.lproj/Localizable.strings

📝 Walkthrough

Walkthrough

Adds localization resource files for Spanish, Quebec French, and French languages. Three new Localizable.strings files introduce translations for UI messages, errors, status updates, and system labels without modifying any code or public APIs.

Changes

Cohort / File(s) Summary
Localization Files
Source/gui/Resources/es.lproj/Localizable.strings, Source/gui/Resources/fr-CA.lproj/Localizable.strings, Source/gui/Resources/fr.lproj/Localizable.strings
Added three new localization files (~665 total lines) providing complete Spanish, Quebec French, and French translations for UI elements, error messages, status updates, and informational text. All files use consistent key-value pair structure with HTML-like formatting in some strings.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the main change: adding translations for French and Spanish localization files to the GUI resources.
Description check ✅ Passed The description correctly relates to the changeset, explaining that Claude Code was used to add internationalization for French, Canadian French, and Spanish.

✏️ 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

Comment @coderabbitai help to get the list of available commands and usage tips.

@pmarkowsky pmarkowsky added this to the 2026.2 milestone Feb 24, 2026
@pmarkowsky
pmarkowsky merged commit f9cd13a into northpolesec:main Feb 24, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/gui Issues or PRs related to the Santa GUI size/l Size: large

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants