Skip to content

[es] Sincroniza traducción de sesiones - #38163

Merged
JuanVqz merged 2 commits into
mdn:mainfrom
MasterIfeanyi:es-sync-django-sessions
Sep 9, 2026
Merged

[es] Sincroniza traducción de sesiones#38163
JuanVqz merged 2 commits into
mdn:mainfrom
MasterIfeanyi:es-sync-django-sessions

Conversation

@MasterIfeanyi

Copy link
Copy Markdown
Contributor

Description

Sincroniza la traducción al español de la página "Django Tutorial Part 7: Sessions framework" con la última versión en inglés. Se actualizó el contenido desactualizado, se limpió el front-matter (solo title, short-title, slug y l10n.sourceCommit) y se corrigieron los enlaces internos para usar el prefijo /es/.

Motivation

Esta página nunca se había registrado para sincronización y llevaba años sin actualizarse respecto a la fuente en inglés, por lo que estaba incompleta y desactualizada para los lectores de habla hispana.

Additional details

Se usó asistencia de IA (Claude) para ayudar con la traducción del contenido; el resultado fue revisado y ajustado manualmente antes de enviarlo.

Related issues and pull requests

Fixes #37245
Relates #9638

@MasterIfeanyi
MasterIfeanyi requested a review from a team as a code owner September 1, 2026 15:42
@MasterIfeanyi
MasterIfeanyi requested review from mariomorillo and removed request for a team September 1, 2026 15:42
@github-actions github-actions Bot added the l10n-es Issues related to Spanish content. label Sep 1, 2026
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Preview URLs (1 page)

External URLs (2)

URL: /es/docs/Learn_web_development/Extensions/Server-side/Django/Sessions
Title: Django Tutorial Parte 7: Framework de sesiones

(comment last updated: 2026-09-09 20:52:56)

@MasterIfeanyi MasterIfeanyi changed the title Sincroniza traducción de sesiones [es] [es] Sincroniza traducción de sesiones Sep 1, 2026
@MasterIfeanyi

Copy link
Copy Markdown
Contributor Author

@JuanVqz please review this

@JuanVqz JuanVqz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@MasterIfeanyi Gracias por tu contribución a la traducción al español de MDN.

Resumen de la revisión

La sincronización es muy buena, pero encontré un punto que rompe el ejercicio en español, así que dejo la revisión como comentario en vez de aprobarla. Es un cambio de una línea (más la frase que la explica).

Verificado:

  • Paridad estructural exacta: 181 líneas, 8 encabezados, 12 vallas de código, 6 bloques y 8 enlaces en ambos archivos, alineados posición por posición; 0 diferencias de indentación.
  • l10n.sourceCommit (48d220a) coincide con el último commit de la fuente en inglés (2025-04-11).
  • short-title traducido ("7: Framework de sesiones"), que es el campo que se suele olvidar y hace que el título se corte en la barra lateral.
  • Front matter correcto: sin page-type ni sidebar, y el slug idéntico al inglés.
  • Bloques de código: los identificadores de Django (num_books, num_instances_available, num_visits) y la sintaxis de plantilla quedan intactos; sólo se tradujo el texto visible.
  • Enlaces: ninguno quedó en /en-US/; los enlaces a la documentación de Django se mantienen igual que en la fuente.
  • Estilo: tuteo consistente, sin espacios sobrantes, sin marcadores TODO, sin unicode invisible.
  • Enlace al issue: #37245 es la sub-tarea correcta de este documento.

Detalle

1. El filtro pluralize no funciona con «vez» (líneas 162 y 166). En inglés time\{{ num_visits|pluralize }} funciona porque el plural sólo agrega una «s». En español el plural de «vez» es irregular, así que vece\{{ num_visits|pluralize }} renderiza:

  • con 1 visita: «Has visitado esta página 1 vece
  • con 2 visitas: «Has visitado esta página 2 veces

O sea que quien siga el tutorial ve una palabra que no existe justo la primera vez que carga la página, que es el momento en que se comprueba que el ejemplo funciona.

Django cubre exactamente este caso: pluralize acepta las dos formas separadas por coma, \{{ valor|pluralize:"singular,plural" }}. Dejo la sugerencia en línea para el bloque de código y otra para la frase que lo explica, que también menciona la «s».

Comment thread files/es/learn_web_development/extensions/server-side/django/sessions/index.md Outdated
Comment thread files/es/learn_web_development/extensions/server-side/django/sessions/index.md Outdated

@JuanVqz JuanVqz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@MasterIfeanyi Aprobado, y gracias por el arreglo. 🎉

Verificado en esta ronda (commit address feedback):

  • El filtro quedó correcto. Comprobé la implementación de pluralize en el código de Django (django/template/defaultfilters.py): devuelve singular_suffix if float(value) == 1 else plural_suffix. Con ve\{{ num_visits|pluralize:"z,ces" }} la página renderiza «1 vez», «2 veces» y también «0 veces», que es lo correcto en español.
  • La frase que lo explica se actualizó en consecuencia, y ya no dice que baste con agregar una «s».
  • En la página renderizada del preview se lee la sintaxis de plantilla tal cual (ve\{{ num_visits|pluralize:"z,ces" }}) y la explicación con «vez» y «veces», así que el escape de las llaves sigue funcionando.
  • Nada más se movió: 181 líneas, 8 encabezados, 12 vallas y 6 bloques siguen alineados con el inglés, con 0 diferencias de indentación. l10n.sourceCommit (48d220a) sigue siendo el último commit de la fuente.
  • Página limpia: flaws: {}, 0 errores de macro, TOC completo con las 8 secciones.
  • Todos los fragmentos entre páginas resuelven contra el sitio en español.
  • CI en verde, y #37245 es la sub-tarea correcta de este documento.

Vale la pena guardarse el caso: es un fallo que no se ve al revisar, porque el código no cambia respecto del inglés y la prosa alrededor se lee bien. Sólo falla para quien lee, y justo en el singular, que es lo primero que aparece al cargar la página por primera vez. Aplica igual a país/países, luz/luces o mes/meses.

@JuanVqz
JuanVqz merged commit b2620b2 into mdn:main Sep 9, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

l10n-es Issues related to Spanish content.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[es] Sincronizar Learn_web_development/Extensions/Server-side/Django/Sessions

2 participants