-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
i18n(es): update nav.ts
#8568
i18n(es): update nav.ts
#8568
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
src/i18n/es/nav.ts
Outdated
'install-and-setup': 'Instalación y Configuración', | ||
'guides/deploy': 'Despliega tu Aplicación', | ||
'guides/migrate-to-astro': 'Migra a Astro', | ||
'blog-tutorial': 'Tutorial: Hacer un Blog', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No sería mejor que dijera Tutorial: Construye un Blog ? Así transmite la misma intención que en su idioma original ... Build a Blog. Qué opinas ? @ItzRoBeerT
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tienes toda la razón!
src/i18n/es/nav.ts
Outdated
|
||
coreConcepts: 'Conceptos Principales', | ||
'concepts/why-astro': 'Por qué Astro', | ||
'coreConcepts': 'Conceptos Básicos', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alguna razón para cambiar Principales por Básicos ? El primero se alinea mejor con su versión en inglés. 🤔
Aunque... ahora que lo veo por 10ma vez, suena igual o mejor jsjs
src/i18n/es/nav.ts
Outdated
'basics/project-structure': 'Estructura del proyecto', | ||
'basics/astro-components': 'Componentes', | ||
'basics/astro-pages': 'Páginas', | ||
'basics/layouts': 'Plantillas', | ||
'basics/astro-syntax': 'Sintaxis de Astro', | ||
'basics/layouts': 'Diseños', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Qué tal "Esquema" en lugar de "Diseños"?
src/i18n/es/nav.ts
Outdated
'guides/middleware': 'Middleware', | ||
'guides/testing': 'Testing', | ||
'guides/images' : 'Imágenes', | ||
'guides/fonts': 'Imágenes', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Acá debería decir Fuentes mi king xd
src/i18n/es/nav.ts
Outdated
'guides/internationalization': 'Internacionalización', | ||
'guides/middleware': 'Middleware', | ||
'guides/testing': 'Testing', | ||
'guides/images' : 'Imágenes', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hay un espacio de más en 'guides/images'[AQUI]: 'Imágenes'
osea no es por reventar las pelotas pero ahí está jsjs
src/i18n/es/nav.ts
Outdated
'guides/images' : 'Imágenes', | ||
'guides/fonts': 'Imágenes', | ||
|
||
'data': 'Conectar Tus Datos', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mejor queda Conecta tus Datos en lugar de "Conectar..." ya que en inglés dice Connect Your Data (lo de "tus" en lugar de "Tus" ya es personal xd creo que queda mejor así en español al no ser un sustantivo y ser palabra corta)
src/i18n/es/nav.ts
Outdated
'data': 'Conectar Tus Datos', | ||
'guides/data-fetching': 'Obtención de Datos', | ||
'guides/astro-db': 'Astro DB', | ||
'guides/backend': 'Añade servicios de Backend', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...Servicios... en lugar de "...servicios..." al ser una palabra más significativa en mi opinión y cuadra con su versión en inglés.
src/i18n/es/nav.ts
Outdated
'guides/rss': 'Agregar un feed RSS', | ||
'guides/view-transitions': 'Transiciones de vista', | ||
|
||
'routes': 'Recursos', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Creo que aquí quisiste escribir 'assets' en lugar de 'routes'
@ItzRoBeerT Hola mi king, vaya mis comentarios se agregaron uno por uno en lugar de todos de golpe 😥 sorry. Dime que opinas de las sugerencias. Algunas son menos relevantes que otras, espero aporte a la calidad de tu PR. Saludos. Por cierto noté que no hay referencia a estos keys de la versión en inglés:
quizás porque actualizaron recientemente la versión en inglés. |
Gracias por los comentarios @juanhuanaco!!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ItzRoBeerT Mi king 👑, ahí agregué unos comentarios finales, revísalos y ya con eso podré dar mi voto de aprobación a tu aporte. Gracias por tu tiempo!
src/i18n/es/nav.ts
Outdated
'basics/project-structure': 'Estructura del proyecto', | ||
'basics/astro-components': 'Componentes', | ||
'basics/astro-pages': 'Páginas', | ||
'basics/layouts': 'Plantillas', | ||
'basics/astro-syntax': 'Sintaxis de Astro', | ||
'basics/layouts': 'Esquema', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
linea 19: Esquemas (en plural, mi error, lo habia escrito en singular, pero dice layouts)
src/i18n/es/nav.ts
Outdated
@@ -1,82 +1,85 @@ | |||
import { NavDictionary } from '../translation-checkers'; | |||
|
|||
export default NavDictionary({ | |||
startHere: 'Empezar Aquí', | |||
'welcomeWorld': '¡Hola, Mundo!', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
linea 4: ¡Hola Mundo! (sin la coma).
Sí, he notado que en la versión en inglés dice Welcome, World! (con coma)
pero en ese caso debería decir "¡Bienvenido, Mundo!" (si quisieramos traducir tal cual).
En mi opinión ¡Hola Mundo! queda mejor al ser la frase más conocida por la comunidad dev para iniciar con una tecnología/lenguaje. ¿ Qué opinas ?
src/i18n/es/nav.ts
Outdated
'guides/markdown-content': 'Markdown y MDX', | ||
'guides/client-side-scripts': 'Scripts y Manejo de Eventos', | ||
'guides/endpoints': 'EndPoints', | ||
'guides/prefetch': 'Prefetch', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
linea: 35 Precarga en lugar de Prefetch quedaría mejor
src/i18n/es/nav.ts
Outdated
'guides/routing': 'Enrutamiento', | ||
'guides/markdown-content': 'Markdown y MDX', | ||
'guides/client-side-scripts': 'Scripts y Manejo de Eventos', | ||
'guides/endpoints': 'EndPoints', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
linea 34: Endpoints (con P minúscula)
src/i18n/es/nav.ts
Outdated
'reference/dev-toolbar-app-reference': 'API de la Barra de Herramientas de Desarrollo', | ||
'reference/directives-reference': 'Directivas de Plantilla', | ||
'reference/cli-reference': 'La CLI de Astro', | ||
'reference/api-reference': 'API de tiempo de Ejecución de Astro', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
linea 63: API de Tiempo de Ejecución de Astro (con T mayúscula en 'tiempo')
src/i18n/es/nav.ts
Outdated
'reference/api-reference': 'API de tiempo de Ejecución de Astro', | ||
'reference/cli-reference': 'CLI de Astro', | ||
'reference/directives-reference': 'Referencia de Directivas', | ||
'guides/typescript': 'Referencia de Typescript', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...TypeScript (S en mayúscula, como en la versión en inglés, forma correcta de escribir la tecnología al hacer referencias).
me había olvidado de ese detalle, pero en los chats le decimos solo typescript no ? jsjsjs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Listo @juanhuanaco 👑!
Muchas gracias por tomarte el tiempo de revisar la PR!
No me había percatado de tantos errores!!
Cualquier otra cosa no dudes en comentar!
Un saludo!❤️
…nto i18n-adding-changes-es
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Buen aporte @ItzRoBeerT 😉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Good work peeps!
Description (required)
Updated
nav.ts
with new Spanish translations. I refactored all content to match the format used innav.ts
from theen
folder. Please let me know if you spot any errors! Happy coding!Related issues & labels (optional)
Discord: @ItzRoBeerT