Skip to content

Added Spanish translation.#113

Merged
wizzomafizzo merged 4 commits into
ZaparooProject:mainfrom
crodriguezdominguez:main
May 20, 2026
Merged

Added Spanish translation.#113
wizzomafizzo merged 4 commits into
ZaparooProject:mainfrom
crodriguezdominguez:main

Conversation

@crodriguezdominguez

@crodriguezdominguez crodriguezdominguez commented May 4, 2026

Copy link
Copy Markdown
Contributor

I have added support to Spanish in the app.

Summary by CodeRabbit

  • New Features

    • Spanish (es-ES) language support is fully integrated. Users can select Spanish in app settings; UI, navigation, messages, errors, onboarding, NFC flows, and tours are translated.
  • Documentation

    • Internationalization guidance updated to note Spanish translations are available and to continue adding new UI strings to the primary English source.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 4, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@wizzomafizzo has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 35 minutes and 27 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 49eb77c3-03d4-45c3-9c7f-4b2019437311

📥 Commits

Reviewing files that changed from the base of the PR and between 2335edc and bf8e394.

📒 Files selected for processing (2)
  • README.md
  • src/routes/settings.about.tsx
📝 Walkthrough

Walkthrough

This pull request adds comprehensive Spanish language support to the application by introducing a new Spanish translation file containing 741 UI strings, registering the es-ES and es locales within the i18n system, exposing Spanish as a selectable language option in the settings page, and documenting the addition in the supported languages list.

Changes

Spanish Language Support

Layer / File(s) Summary
Translation Resources
src/translations/es-ES.json, src/i18n.ts
New Spanish translation file with comprehensive UI strings; es-ES (and es) imported/registered in i18n resources.
Settings UI & Docs
src/routes/settings.index.tsx, AGENTS.md
Settings language selector maps eses-ES and adds Español option; AGENTS.md updated to reference src/translations/es-ES.json.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🐰 ¡Hola, amigos! Our bunny hops with Spanish delight,
Translations bloom and settings shine so bright,
From i18n config to UI just right,
Seven hundred strings take flight,
¡Bienvenidos! The app now speaks español! 🌻

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Added Spanish translation' clearly and directly summarizes the main change in the pull request, which is to add Spanish language support across the application.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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 coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 3

🤖 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/routes/settings.index.tsx`:
- Line 176: Update the language dropdown entry that currently renders the option
with value "es-ES" so its visible label uses the native name "Español" instead
of the English "Spanish"; locate the option element with value="es-ES" in the
settings UI (in settings.index.tsx) and replace the display text while leaving
the value attribute intact to preserve form behavior.

In `@src/translations/es-ES.json`:
- Line 371: Three translation keys remain in English; update the Spanish locale
values for online.agreementSignUp, settings.updateDb.progressLabel, and
settings.logs.uploadError to the provided Spanish strings ("Al pulsar
registrarse o acceder, acepta nuestros", "Progreso de actualización de la base
de datos", "Fallo al subir los logs") so the file src/translations/es-ES.json
returns Spanish text for those keys before shipping.
- Line 18: Update the four Spanish strings in src/translations/es-ES.json:
change the value for key "yes" from "Si" to "Sí"; change
"settings.updateDb.status.ready" from "La base de datos esta lista" to "La base
de datos está lista"; change "settings.readers.noReadersDetected" from "No se
han encontrados lectores" to "No se han encontrado lectores"; and change
"tagSelector.noTags" from "No hay disponible etiquetas" to "No hay etiquetas
disponibles".
🪄 Autofix (Beta)

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: defaults

Review profile: CHILL

Plan: Pro

Run ID: aa0cce99-2457-4183-a9f0-dd4c5ad933c5

📥 Commits

Reviewing files that changed from the base of the PR and between 7c304a4 and 5c44486.

📒 Files selected for processing (4)
  • AGENTS.md
  • src/i18n.ts
  • src/routes/settings.index.tsx
  • src/translations/es-ES.json

Comment thread src/routes/settings.index.tsx Outdated
Comment thread src/translations/es-ES.json Outdated
Comment thread src/translations/es-ES.json Outdated
@codecov

codecov Bot commented May 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@wizzomafizzo
wizzomafizzo merged commit bd6dcc6 into ZaparooProject:main May 20, 2026
5 checks passed
@wizzomafizzo

Copy link
Copy Markdown
Member

@crodriguezdominguez thank you so much!

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.

2 participants