Conversation
Hi! , just a bunch of little changes, looking for misplaced caps.
Codecov Report✅ All modified and coverable lines are covered by tests. 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:
|
VREMSoftwareDevelopment
left a comment
There was a problem hiding this comment.
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):
permission_msg:¡Puedeshas a capitalPafter a comma. Adding the opening¡is right (the original only had the closing!), but the word should be lowercase.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". Usingsección de preguntas frecuentesalso matchesabout_faq(Preguntas frecuentes:).
Optional, not blocking:
channel_rating_best_alternative:trataris 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> |
There was a problem hiding this comment.
The English source ("Please visit frequently ask questions section") means the FAQ section. Suggest making that explicit, which also matches about_faq:
| <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> |
There was a problem hiding this comment.
¡Puedes should be lowercase after the comma (, ¡puedes):
| <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> |
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Three Spanish wording and capitalization issues remain unresolved.
Review effort: Lite
Findings: 2
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> |
|
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. |

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