Skip to content

fix: fixes spelling errors - #172

Merged
fsologureng merged 1 commit into
eol/koa.masterfrom
cataIlabaca/38-dev-translations
Aug 6, 2025
Merged

fix: fixes spelling errors#172
fsologureng merged 1 commit into
eol/koa.masterfrom
cataIlabaca/38-dev-translations

Conversation

@catalinaIlabaca

@catalinaIlabaca catalinaIlabaca commented Jan 17, 2025

Copy link
Copy Markdown

Se actualizan los archivos de traducción para modificar los textos según lo solicitado

Se modifica la palabra enlaze y las sugerencias dadas por Thomas

@fsologureng fsologureng 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.

No revisé más que hasta el cambio de msgid.

Por otro lado, si te fijas en los commits anteriores (https://github.com/eol-uchile/edx-platform/commits/eol/koa.master/) el tratamiento en EOL es de tu, no de Ud. Hay mucho por hacer porque el es_419 upstream es de Ud.

#: lms/djangoapps/courseware/date_summary.py
msgid ""
"You are still eligible to upgrade to a Verified Certificate! Pursue it to "
"¡You are still eligible to upgrade to a Verified Certificate! Pursue it to "

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No se puede cambiar el msgid!

Comment thread conf/locale/es_419/LC_MESSAGES/django.po Outdated
Comment thread conf/locale/es_419/LC_MESSAGES/django.po Outdated
Comment thread conf/locale/es_419/LC_MESSAGES/django.po Outdated
Comment thread conf/locale/es_419/LC_MESSAGES/django.po Outdated
Comment thread conf/locale/es_419/LC_MESSAGES/djangojs.po Outdated
Comment thread conf/locale/es_419/LC_MESSAGES/djangojs.po Outdated
Comment thread conf/locale/es_419/LC_MESSAGES/djangojs.po Outdated
Comment thread conf/locale/es_419/LC_MESSAGES/djangojs.po Outdated
@catalinaIlabaca catalinaIlabaca linked an issue Jul 9, 2025 that may be closed by this pull request
@catalinaIlabaca
catalinaIlabaca force-pushed the cataIlabaca/38-dev-translations branch from 02ff1d5 to 26532d1 Compare July 9, 2025 15:17
@catalinaIlabaca

Copy link
Copy Markdown
Author

Hice force push para actualizar la rama a koa.master

Comment thread conf/locale/es_419/LC_MESSAGES/django.po Outdated
Comment thread conf/locale/es_419/LC_MESSAGES/django.po Outdated
Comment thread conf/locale/es_419/LC_MESSAGES/djangojs.po Outdated
msgstr ""
"Un correo electrónico ha sido enviado a su {userEmail} con un enlaze para "
"Un correo electrónico ha sido enviado a su {userEmail} con un enlace para "
"activar su cuenta."

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

"tu cuenta" (si no la corrección de Thomas va a quedar inconsistente)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

E insisto con usar casilla en español, ya que en inglés sería como:

  • An email has been sent to tomaspemora@uchile.cl with a ...

Pero en español, usar el adjetivo "tu" no corresponde en esa frase.

  • Un correo electrónico ha sido enviado a tu tomaspemora@uchile.cl con ...

Podría ser de 2 formas:

  • Un correo electrónico ha sido enviado a tomaspemora@uchile.cl con ...
  • Un correo electrónico ha sido enviado a tu casilla tomaspemora@uchile.cl con ...

Prefiero la 2da.

@tomaspemora

Copy link
Copy Markdown

pasamos la posta de este PR de @catalinaIlabaca a @vicente-astorga

Comment thread cms/static/js/i18n/es-419/djangojs.js Outdated
Comment thread cms/static/js/i18n/es-419/djangojs.js Outdated
Comment thread conf/locale/es_419/LC_MESSAGES/djangojs.po Outdated
msgstr ""
"Un correo electrónico ha sido enviado a su {userEmail} con un enlaze para "
"Un correo electrónico ha sido enviado a su {userEmail} con un enlace para "
"activar su cuenta."

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

E insisto con usar casilla en español, ya que en inglés sería como:

  • An email has been sent to tomaspemora@uchile.cl with a ...

Pero en español, usar el adjetivo "tu" no corresponde en esa frase.

  • Un correo electrónico ha sido enviado a tu tomaspemora@uchile.cl con ...

Podría ser de 2 formas:

  • Un correo electrónico ha sido enviado a tomaspemora@uchile.cl con ...
  • Un correo electrónico ha sido enviado a tu casilla tomaspemora@uchile.cl con ...

Prefiero la 2da.

Comment thread lms/static/js/i18n/es-419/djangojs.js Outdated
Comment thread lms/static/js/i18n/es-419/djangojs.js Outdated
@fsologureng

fsologureng commented Aug 4, 2025

Copy link
Copy Markdown

@tomaspemora

E insisto con usar casilla en español, ya que en inglés sería como:

An email has been sent to tomaspemora@uchile.cl with a ...

Pero en español, usar el adjetivo "tu" no corresponde en esa frase.

Un correo electrónico ha sido enviado a tu tomaspemora@uchile.cl con ...

Podría ser de 2 formas:

Un correo electrónico ha sido enviado a tomaspemora@uchile.cl con ...
Un correo electrónico ha sido enviado a tu casilla tomaspemora@uchile.cl con ...

Prefiero la 2da.

Creo que se confunde un poco el objetivo de la traducción; Agregar tu casilla sería consistente con que el original fuese An email has been sent to your {userEmail} mailbox with a link for you to activate your account., pero no es así.
El tema es que el castellano upstream está mal traducido incluso con ese "su" que se le agrega.
Yo opino que la traducción correcta con el tratamiento en 2da persona es Se ha enviado un correo electrónico a {userEmail} con un enlace para que actives tu cuenta. (DeepL)

@tomaspemora

tomaspemora commented Aug 4, 2025

Copy link
Copy Markdown

@tomaspemora

E insisto con usar casilla en español, ya que en inglés sería como:
An email has been sent to tomaspemora@uchile.cl with a ...
Pero en español, usar el adjetivo "tu" no corresponde en esa frase.
Un correo electrónico ha sido enviado a tu tomaspemora@uchile.cl con ...
Podría ser de 2 formas:
Un correo electrónico ha sido enviado a tomaspemora@uchile.cl con ...
Un correo electrónico ha sido enviado a tu casilla tomaspemora@uchile.cl con ...
Prefiero la 2da.

Creo que se confunde un poco el objetivo de la traducción; Agregar tu casilla sería consistente con que el original fuese An email has been sent to your {userEmail} mailbox with a link for you to activate your account., pero no es así. El tema es que el castellano upstream está mal traducido incluso con ese "su" que se le agrega. Yo opino que la traducción correcta con el tratamiento en 2da persona es Se ha enviado un correo electrónico a {userEmail} con un enlace para que actives tu cuenta. (DeepL)

Perfecto, estoy de acuerdo.

Como punto aparte, creo que hay harto que podemos aportar en cuanto a las traducciones del español en openedx, pero es muuuuuuucha pega.

@fsologureng

Copy link
Copy Markdown

Como punto aparte, creo que hay harto que podemos aportar en cuanto a las traducciones del español en openedx, pero es muuuuuuucha pega.

Muy de acuerdo, hay mucho string en pésimo estado de traducción, me llama la atención que habiendo tanto proveedor latinoamericano top contributor en el ecosistema, sean tan malas las traducciones.

@vicente-astorga

vicente-astorga commented Aug 4, 2025

Copy link
Copy Markdown

Corregí las revisiones que dejaron. Estoy siguiendo según el objetivo de cambiar de persona, pero hay harto por corregir. Me pregunto si hay alguna prioridad, o si avanzo según identificación de errores general.

Comment thread cms/static/js/i18n/es-419/djangojs.js Outdated
Comment thread cms/static/js/i18n/es-419/djangojs.js Outdated
Comment thread conf/locale/es_419/LC_MESSAGES/django.po Outdated
Comment thread conf/locale/es_419/LC_MESSAGES/django.po Outdated
Comment thread conf/locale/es_419/LC_MESSAGES/djangojs.po Outdated
Comment thread conf/locale/es_419/LC_MESSAGES/djangojs.po Outdated
Comment thread lms/static/js/i18n/es-419/djangojs.js
Comment thread lms/static/js/i18n/es-419/djangojs.js Outdated

@fsologureng fsologureng 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.

Por favor re-revisar antes de pedir review.

Comment thread cms/static/js/i18n/es-419/djangojs.js Outdated
Comment thread conf/locale/es_419/LC_MESSAGES/django.po Outdated

@tomaspemora tomaspemora 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.

Veo bien todos los string donde hicimos correcciones. 👍

@fsologureng

Copy link
Copy Markdown

@vicente-astorga squash por favor

- Fixed spelling and formatting errors
- Improved translation clarity
- Adjusted second person and exclamation marks
@vicente-astorga
vicente-astorga force-pushed the cataIlabaca/38-dev-translations branch from e819789 to ca0e629 Compare August 5, 2025 21:33
@fsologureng
fsologureng added this pull request to the merge queue Aug 6, 2025
Merged via the queue into eol/koa.master with commit 48a0ce7 Aug 6, 2025
7 of 8 checks passed
@catalinaIlabaca
catalinaIlabaca deleted the cataIlabaca/38-dev-translations branch May 14, 2026 19:02
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.

corregir la ortografia de enlaze

4 participants