bito-code-review[bot] commented on code in PR #41609:
URL: https://github.com/apache/superset/pull/41609#discussion_r3503805964
##########
superset/translations/es/LC_MESSAGES/messages.po:
##########
@@ -1012,12 +1020,16 @@ msgstr "El informe se ha creado"
msgid "API key name is required"
msgstr "El nombre del rol es obligatorio"
Review Comment:
<div>
<div id="suggestion">
<div id="issue"><b>Wrong translation cross-contaminated</b></div>
<div id="fix">
Translation for msgid "API key name is required" reads "El nombre del rol es
obligatorio" ('The role name is required') — a cross-contamination error from a
different entry. Should be "El nombre de la clave API es obligatorio.".
</div>
</div>
<small><i>Code Review Run #3e581f</i></small>
</div>
---
Should Bito avoid suggestions like this for future reviews? (<a
href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>)
- [ ] Yes, avoid them
##########
superset/translations/es/LC_MESSAGES/messages.po:
##########
@@ -2792,11 +2826,17 @@ msgstr "Plantillas CSS"
msgid "CSS applied to the chart"
msgstr "CSS aplicado al gráfico"
+# Machine-translated via backfill_po.py (claude-sonnet-4-6) [refs: de, sr,
+# sr_Latn]
+#, fuzzy
msgid ""
"CSS styles may be removed by server-side HTML sanitization. If styles are"
" not applying, ask your Superset administrator to adjust the HTML "
"sanitization configuration."
msgstr ""
+"Los estilos CSS pueden ser eliminados por la depuración HTML del lado del"
+" servidor. Si los estilos no se aplican, solicite al administrador de "
+"Superset que ajuste la configuración de depuración HTML."
Review Comment:
<div>
<div id="suggestion">
<div id="issue"><b>CWE-117: Translation Error on Security Concept</b></div>
<div id="fix">
The term 'depuración HTML' (HTML debugging) is an incorrect translation of
'HTML sanitization' (a security feature that strips dangerous HTML/CSS). In
Spanish security terminology, this should be 'desinfección' or 'sanitización'
to convey the correct meaning to administrators investigating why their CSS
styles are removed. (See also:
[CWE-117](https://cwe.mitre.org/data/definitions/117.html))
</div>
</div>
<small><i>Code Review Run #3e581f</i></small>
</div>
---
Should Bito avoid suggestions like this for future reviews? (<a
href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>)
- [ ] Yes, avoid them
##########
superset/translations/es/LC_MESSAGES/messages.po:
##########
@@ -16503,8 +16816,11 @@ msgstr "Desanclar"
msgid "Unpin from the result panel"
msgstr "Fijar en el panel de resultados"
Review Comment:
<div>
<div id="suggestion">
<div id="issue"><b>Semantic error: opposite-direction verb</b></div>
<div id="fix">
The msgstr says "Fijar" (to pin/fix), which is the opposite of the source
"Unpin" (to remove a pin). The file's own precedent uses "Desanclar" for
"Unpin" (line 16812) and "Fijar" for "Pin". This was a backfill-generated
translation that needs correction before the fuzzy flag is removed.
</div>
</div>
<small><i>Code Review Run #3e581f</i></small>
</div>
---
Should Bito avoid suggestions like this for future reviews? (<a
href=https://alpha.bito.ai/home/ai-agents/review-rules>Manage Rules</a>)
- [ ] Yes, avoid them
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
