Skip to content

Update Spanish strings - #624

Closed
dtorner wants to merge 1 commit into
VREMSoftwareDevelopment:mainfrom
dtorner:patch-1
Closed

dtorner wants to merge 1 commit into
VREMSoftwareDevelopment:mainfrom
dtorner:patch-1

Conversation

@dtorner

@dtorner dtorner commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

Hi! , just a bunch of little changes, looking for misplaced caps.

Hi! , just a bunch of little changes, looking for misplaced caps.
@codecov

codecov Bot commented Sep 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.09%. Comparing base (198a70b) to head (0a6c3b3).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #624   +/-   ##
=========================================
  Coverage     98.09%   98.09%           
  Complexity     1025     1025           
=========================================
  Files           127      127           
  Lines          2940     2940           
  Branches        248      248           
=========================================
  Hits           2884     2884           
  Misses           16       16           
  Partials         40       40           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@VREMSoftwareDevelopment VREMSoftwareDevelopment left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks for the cleanup. The sentence-case fixes and the move to informal "tú" look good and make the file consistent (other strings such as Optimiza tu red and Mientras la utilizas… tu ubicación already use "tú"). Placeholders (%1$s, %2$s) and keys are untouched, and the file is still valid UTF-8.

Two small changes requested (inline suggestions):

  1. permission_msg: ¡Puedes has a capital P after a comma. Adding the opening ¡ is right (the original only had the closing !), but the word should be lowercase.
  2. no_data_msg: the English source ("Please visit frequently ask questions section") is a garbled "FAQ section". The current text still reads as "visit frequently". Using sección de preguntas frecuentes also matches about_faq (Preguntas frecuentes:).

Optional, not blocking:

  • channel_rating_best_alternative: tratar is a bit awkward, e.g. %1$s, prueba la banda Wi-Fi alternativa %2$s.
  • Follow-ups for a separate PR: Librerias → Librerías, Esconder → Ocultar, Lenguaje → Idioma.

CI: everything is green except emulator-test (35), which failed in connectedDebugAndroidTest with an emulator error (bad color buffer handle) and looks unrelated to a strings-only change. A re-run should clear it.

<string name="no_data">"Sin datos"</string>
<string name="no_data_msg">"Por favor, visite frecuentemente preguntas sección en nuestro sitio web."</string>
<string name="permission_msg">"A partir de Android Marshmallow, se requieren permisos de ubicación para realizar un escaneo Wi-Fi. Se le pedirá que habilite los servicios / permisos de ubicación. Como esta aplicación no requiere acceso a Internet, puede estar seguro de que esta aplicación no puede transmitir su ubicación!"</string>
<string name="no_data_msg">"Por favor, visita frecuentemente la sección de preguntas de nuestro sitio web."</string>

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

The English source ("Please visit frequently ask questions section") means the FAQ section. Suggest making that explicit, which also matches about_faq:

Suggested change
<string name="no_data_msg">"Por favor, visita frecuentemente la sección de preguntas de nuestro sitio web."</string>
<string name="no_data_msg">"Por favor, visita la sección de preguntas frecuentes de nuestro sitio web."</string>

<string name="no_data_msg">"Por favor, visite frecuentemente preguntas sección en nuestro sitio web."</string>
<string name="permission_msg">"A partir de Android Marshmallow, se requieren permisos de ubicación para realizar un escaneo Wi-Fi. Se le pedirá que habilite los servicios / permisos de ubicación. Como esta aplicación no requiere acceso a Internet, puede estar seguro de que esta aplicación no puede transmitir su ubicación!"</string>
<string name="no_data_msg">"Por favor, visita frecuentemente la sección de preguntas de nuestro sitio web."</string>
<string name="permission_msg">"A partir de Android Marshmallow, se requieren permisos de ubicación para realizar un escaneo Wi-Fi. Se te pedirá que habilites los servicios / permisos de ubicación. Como esta aplicación no requiere acceso a Internet, ¡Puedes estar seguro de que esta aplicación no puede transmitir tu ubicación!"</string>

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

¡Puedes should be lowercase after the comma (, ¡puedes):

Suggested change
<string name="permission_msg">"A partir de Android Marshmallow, se requieren permisos de ubicación para realizar un escaneo Wi-Fi. Se te pedirá que habilites los servicios / permisos de ubicación. Como esta aplicación no requiere acceso a Internet, ¡Puedes estar seguro de que esta aplicación no puede transmitir tu ubicación!"</string>
<string name="permission_msg">"A partir de Android Marshmallow, se requieren permisos de ubicación para realizar un escaneo Wi-Fi. Se te pedirá que habilites los servicios / permisos de ubicación. Como esta aplicación no requiere acceso a Internet, ¡puedes estar seguro de que esta aplicación no puede transmitir tu ubicación!"</string>

Copilot AI 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.

Copilot review overview

🟡 Changes recommended

Three Spanish wording and capitalization issues remain unresolved.

Review effort: Lite
Findings: 2 Low severity

Open (2)
What changed in this PR

Updates Spanish localization strings for capitalization and wording clarity.

Changes:

  • Corrects sentence-case capitalization.
  • Improves instructional, permission, and Wi-Fi terminology.
File Summary
app/​src/​main/​res/​values-es/​strings.xml Updated Spanish UI strings.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

<string name="no_data">"Sin datos"</string>
<string name="no_data_msg">"Por favor, visite frecuentemente preguntas sección en nuestro sitio web."</string>
<string name="permission_msg">"A partir de Android Marshmallow, se requieren permisos de ubicación para realizar un escaneo Wi-Fi. Se le pedirá que habilite los servicios / permisos de ubicación. Como esta aplicación no requiere acceso a Internet, puede estar seguro de que esta aplicación no puede transmitir su ubicación!"</string>
<string name="no_data_msg">"Por favor, visita frecuentemente la sección de preguntas de nuestro sitio web."</string>
<string name="no_data_msg">"Por favor, visite frecuentemente preguntas sección en nuestro sitio web."</string>
<string name="permission_msg">"A partir de Android Marshmallow, se requieren permisos de ubicación para realizar un escaneo Wi-Fi. Se le pedirá que habilite los servicios / permisos de ubicación. Como esta aplicación no requiere acceso a Internet, puede estar seguro de que esta aplicación no puede transmitir su ubicación!"</string>
<string name="no_data_msg">"Por favor, visita frecuentemente la sección de preguntas de nuestro sitio web."</string>
<string name="permission_msg">"A partir de Android Marshmallow, se requieren permisos de ubicación para realizar un escaneo Wi-Fi. Se te pedirá que habilites los servicios / permisos de ubicación. Como esta aplicación no requiere acceso a Internet, ¡Puedes estar seguro de que esta aplicación no puede transmitir tu ubicación!"</string>
@dtorner

dtorner commented Sep 29, 2026

Copy link
Copy Markdown
Contributor Author

Wow, just a minor language modification, and I'm bombed with a lot of non human slop. Sorry to everybody, but I prefer to close the request.

It has been a pleasure.

David.

@dtorner dtorner closed this Sep 29, 2026
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.

3 participants