Skip to content

feat(i18n): settings en ingles (tarea 12, 18/n) - #72

Merged
keilynrp merged 1 commit into
masterfrom
feat/i18n-en-settings
Oct 7, 2026
Merged

keilynrp merged 1 commit into
masterfrom
feat/i18n-en-settings

Conversation

@keilynrp

@keilynrp keilynrp commented Oct 7, 2026

Copy link
Copy Markdown
Collaborator

Resumen

Tarea 12 del plan de i18n, decimoctava parte: traduce a inglés el espacio settings, que usa la página de Ajustes con sus cinco pestañas (arquitectura, estado del sistema, notificaciones, general y equipo). Con common, errors, auth, layout, dashboard, analytics, companyProfile, customerPortal, contracts, crm, discovery, hr, inventory, invoicing, portalBuilder, projects y sales son 18 de 19; en sigue sin ofrecerse a nadie hasta que estén todos (releasedLocales).

  • Se registra en TRANSLATED, así que los checks de paridad (claves y marcadores) y de cadenas copiadas del español lo cubren desde ya.
  • Once cadenas que son iguales de verdad se aprueban una a una en IDENTICAL_TO_BASE: General, CRM, PostgreSQL, Event Bus (dos veces), API Gateway (tres), Auth · Rate Limit · Routing y v{{version}}. Eran exactamente las once que señaló el check.
  • Ortografía de EE. UU., como se acordó en feat(i18n): crm en ingles (tarea 12, 10/n) #64.

Tests con inglés real (english-settings.test.tsx)

No usan el catálogo pseudo-localizado de Settings.test.tsx, sino el catálogo en que se envía:

  • Cabecera y las cinco pestañas.
  • El diagrama de arquitectura (capas, nodos y sus subtítulos) y las tres tarjetas bajo él.
  • Estado del sistema: cada servicio con su estado y la latencia, el estado general, la versión, la frescura del dato y el aviso de backend inalcanzable.
  • Notificaciones: tipos de evento con interruptores con nombre y su resumen, el guardado de un interruptor, el interruptor general y la tarjeta de notificaciones en tiempo real.
  • General y equipo: el formulario del workspace, las preferencias y los roles.

Verificación

  • Mutaciones sobre el catálogo real: un texto copiado del español, un marcador {{active}} perdido y una clave perdida hacen fallar el check de catálogos y un test de la página; restaurado después.
  • Typecheck y lint limpios.
  • Suite completa del frontend: 50 ficheros, 757 tests.
  • Backend sin cambios (los recuentos documentados no varían).

Siguiente

Queda 1 espacio: workflows.

🤖 Generated with Claude Code

Tarea 12 del plan, decimoctava parte: traduce a `en` el espacio `settings`, que usa la
pagina de Ajustes con sus cinco pestanas (arquitectura, estado del sistema,
notificaciones, general y equipo). Con `common`, `errors`, `auth`, `layout`,
`dashboard`, `analytics`, `companyProfile`, `customerPortal`, `contracts`, `crm`,
`discovery`, `hr`, `inventory`, `invoicing`, `portalBuilder`, `projects` y `sales` son
18 de 19; `en` sigue sin ofrecerse hasta que esten todos.

Se registra en `TRANSLATED`, asi que los checks de paridad (claves y marcadores) y de
cadenas copiadas lo cubren. Once cadenas iguales de verdad se aprueban una a una en
`IDENTICAL_TO_BASE`: `General`, `CRM`, `PostgreSQL`, `Event Bus` (dos veces),
`API Gateway` (tres), `Auth · Rate Limit · Routing` y `v{{version}}`. Ortografia de
EE. UU.

Tests en ingles real (`english-settings.test.tsx`): cabecera y pestanas, el diagrama de
arquitectura y sus tres tarjetas, el estado de cada servicio con la latencia, el estado
general, la version y la frescura del dato, el aviso de backend inalcanzable, los tipos
de evento con interruptores con nombre y su resumen, el interruptor general, el
formulario del workspace y los roles del equipo.

Comprobado con mutaciones sobre el catalogo real: un texto copiado del espanol, un
marcador perdido y una clave perdida fallan el test.

Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-07T05:52:26.786431Z 9cc5c43 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9cc5c43ead

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +58 to +60
"services": {
"api": "API Gateway",
"postgres": "PostgreSQL"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Add the Redis service label to the translated catalogue

When this page uses the real /health response, backend/app/health.py always includes a third check named redis (lines 86–89), but the new English service map only covers api and postgres. useEnumLabel therefore falls back to the wire identifier and renders lowercase redis, even though this namespace is now marked fully translated; add the Redis label to both base and English catalogues and cover the actual three-service response in the English test.

Useful? React with 👍 / 👎.

@keilynrp
keilynrp merged commit 425b46a into master Oct 7, 2026
5 checks passed
@keilynrp
keilynrp deleted the feat/i18n-en-settings branch October 7, 2026 06:26
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.

1 participant