Skip to content

feat(i18n): Spanish (es) locale (#126) with review fixes - #133

Merged
jp1337 merged 3 commits into
mainfrom
i18n/spanish
Oct 8, 2026
Merged

jp1337 merged 3 commits into
mainfrom
i18n/spanish

Conversation

@jp1337

@jp1337 jp1337 commented Oct 8, 2026

Copy link
Copy Markdown
Member

Warning

AI-generated

Carries @edumicro's #126 unchanged (its commits, authorship kept) plus one review commit, so the required CodeQL check runs: GitHub starts no CodeQL run for that fork PR. Merging this marks #126 merged.

Review commit:

  • the home page trust strip said "4 languages"; a test now ties that count, the locale files and _SUPPORTED together (red with the old text);
  • app/models/category.py comment lists es;
  • nav.lang_switch_label_* dropped from all five locales: no template reads them.

Review of #126: 780/780 keys, formal usted, placeholders and markup identical to en.json; 137 i18n tests green.

🤖 Generated with Claude Code

https://claude.ai/code/session_013NwHs21bPEE5cpr4rDSeZz

edumicro and others added 3 commits October 5, 2026 14:24
Full es.json (Spain Spanish, formal usted), added to the language picker,
Accept-Language negotiation, /set-language allowlist and the per-locale
thousands separator. Docs and README that list the UI languages now
include Spanish.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…bels

Review of #126: the home page trust strip still said four languages, and a test now ties that count, the locale files and the supported set together. The nav.lang_switch_label_* keys were read by no template in any locale.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013NwHs21bPEE5cpr4rDSeZz
@github-actions github-actions Bot added the ai-generated Says it was written by an AI agent (AGENTS.md) label Oct 8, 2026
@jp1337 jp1337 mentioned this pull request Oct 8, 2026
2 tasks done
@codecov

codecov Bot commented Oct 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@jp1337
jp1337 merged commit f069f67 into main Oct 8, 2026
15 checks passed
@jp1337
jp1337 deleted the i18n/spanish branch October 8, 2026 20:43
@jp1337 jp1337 mentioned this pull request Oct 9, 2026
3 tasks done
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-generated Says it was written by an AI agent (AGENTS.md)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants