Skip to content
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

translate: adding interactivity to spanish #698

Merged
merged 6 commits into from
May 24, 2023
Merged

Conversation

joelnbl
Copy link
Contributor

@joelnbl joelnbl commented May 17, 2023

  • Adding translation to interactivity to Spanish

  • Updating interactivity sidebar text to Spanish

@carburo
Copy link
Member

carburo commented May 17, 2023

¡Hola, @joelnbl! Gracias por la contribución. Ten en cuenta para futuras traducciones ten en cuenta que primero debe dejarse un comentario en #500 para evitar que dos o más personas terminen traduciendo el mismo artículo. Lee las instrucciones para traductores nuevos en ese mismo issue y comprueba que has cumplido con la guía de estilo y ciertas elecciones de vocabulario que están estandarizadas en este sitio. ¡Muchas gracias!

@joelnbl
Copy link
Contributor Author

joelnbl commented May 17, 2023

Sigue todos los estándares

@joelnbl
Copy link
Contributor Author

joelnbl commented May 19, 2023

@carburo what's the status?

@carburo
Copy link
Member

carburo commented May 23, 2023

Hi! A review from one of the maintainers is still pending before we can merge it. I, personally, haven't had the time to take a look at it yet. Sorry for the inconvenience.

@github-actions
Copy link

github-actions bot commented May 24, 2023

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@joelnbl
Copy link
Contributor Author

joelnbl commented May 24, 2023

@carburo there is an error with the site lint

Copy link
Member

@carburo carburo left a comment

Choose a reason for hiding this comment

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

¡Hola! ¡Gracias por tu contribución! Además de las sugerencias de código hay algunos elementos que deberías revisar antes de que podamos mezclar este PR.

  • El uso de «usted» para la segunda persona del singular. Para mantener la consistencia en la documentación y porque es menos propenso a ambigüedades hemos optado por usar siempre «tú» (Puedes consultar nuestra guía de traducción para otros aspectos como este).
  • Los títulos de las secciones deben coincidir con el de la página a la que referencian. Por ejemplo: la sección se titula «Respondiendo a eventos» mientras el título de la página es «Responder a eventos».
  • Array no se traduce (ver https://github.com/reactjs/es.react.dev/wiki/Glosario)

src/content/learn/adding-interactivity.md Outdated Show resolved Hide resolved
src/content/learn/adding-interactivity.md Outdated Show resolved Hide resolved
src/content/learn/adding-interactivity.md Outdated Show resolved Hide resolved
src/content/learn/adding-interactivity.md Outdated Show resolved Hide resolved
src/content/learn/adding-interactivity.md Outdated Show resolved Hide resolved
@joelnbl
Copy link
Contributor Author

joelnbl commented May 24, 2023

@carburo fixed!

@carburo
Copy link
Member

carburo commented May 24, 2023

@joelnbl Por favor, haz una revisión completa del documento. Aún quedan conjugaciones con la forma de «usted». También haz la revisión de todas las secciones. Sin hacer una revisión exhaustiva puedo ver que hay una sección que se llama «Renderizar y confirmar» y la página «Renderizado y confirmación»

@joelnbl
Copy link
Contributor Author

joelnbl commented May 24, 2023

@carburo ready for review

@joelnbl joelnbl requested a review from carburo May 24, 2023 23:20
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.

2 participants