fix(i18n): es-AR voseo overrides for every second-person app string - #2617
fix(i18n): es-AR voseo overrides for every second-person app string#2617jjramirezn wants to merge 3 commits into
Conversation
The es-AR overlay is deltas-only over es-419, but it carried just 14 entries — so ~650 second-person strings rendered with tuteo conjugation (puedes, inténtalo, escanea) for Argentine users instead of voseo (podés, intentalo, escaneá). Reviewed every string in es-419 with two detector passes (common tú forms + accent-clitic imperatives) and added the missing 651 overrides. Placeholders, ICU plurals and rich-text tags are byte-identical to es-419; only conjugation, pronouns (tú/ti → vos) and AR verb choice (saltarse → saltearse) change.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughChangesArgentine Spanish localization
Estimated code review effort: 2 (Simple) | ~10 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
@coderabbitai review |
Code-analysis diffPainscore total: 7061.25 → 7061.25 (0) |
✅ Action performedReview finished.
|
🧪 UI test report — ✅ all greenSuites
📊 Coverage (unit)
⏱ 10 slowest test cases
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/i18n/app/messages/es-AR.json`:
- Line 22: Update the description value in the Spanish-AR message catalog to
spell the product name as “Mercado Pago,” matching the established usage
elsewhere in the catalog.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 41f9b448-1944-408c-9028-5b2cebd8ccb5
📒 Files selected for processing (1)
src/i18n/app/messages/es-AR.json
Summary
The in-app es-AR catalog is deltas-only over es-419, but the overlay carried just 14 entries. Every other second-person string — ~650 of them — rendered with tuteo conjugation (
puedes,inténtalo,escanea,tú) for Argentine users instead of voseo (podés,intentalo,escaneá,vos). This is exactly what the overlay exists for per its own design note inmessages.ts.This PR adds the missing 651 voseo overrides to
src/i18n/app/messages/es-AR.json(14 → 665 entries).Method (scripted + reviewed):
inténtalo,únete,pídele) and rarer present-tense forms (crees,consigues). Final catch-all sweep returns zero.src/i18n/es-ar.json) audited too: already fully voseo, untouched.Beyond conjugation, deliberately minimal:
tú/ti/contigo→vos/con vos,saltarse→saltearse(AR idiom),has hecho→hiciste(AR pretérito). No terminology changes (dinero vs plata etc. left as-is).Risks / breaking changes
campaign-maps.test.tsback-merge artifact) — unblocker in fix(test): dev tip typecheck break — singular classifyBareCampaign call sites #2616, or superseded by feat(badges): consume declarative acquisition platform #2613.QA
npx jest --testPathPattern src/i18n— 6 suites, 70/70 pass (overlay keys validated against en.json).origin/dev).Screenshots: N/A-ish (copy-only; no layout change) — every changed string is a text swap in place.
🤖 Generated with Claude Code
Summary by CodeRabbit