-
Notifications
You must be signed in to change notification settings - Fork 293
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 input reference #641
Conversation
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.
¡Hola! ¡Gracias por tu contribución! Dejé algunas sugerencias de cambios que puedes aplicar o comentar si no estás de acuerdo. Sin embargo, esta es una revisión preliminar. No he terminado de revisar porque he notado algunas cuestiones que se repiten en varios lugares del documento y que nos ayudaría si las revisaras primero:
- 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).
- Uso de prop como nombre masculino en lugar de femenino como se indica en la guía de traducción (como acortamiento de propiedad).
- Se repiten algunas erratas y errores que podrían ser detectadas por una herramienta automática (por ejemplo, para VSCode puedes usar esta).
Una vez que termines puedes solicitar una nueva revisión.
Gracias por la sugerencias. He arreglado los typos que señalaste y otros que se me pasaron la primera vez. |
Size Changes📦 Next.js Bundle AnalysisThis analysis was generated by the next.js bundle analysis action 🤖 This PR introduced no changes to the javascript bundle 🙌 |
¡Gracias! |
Hola. Estoy atento a cualquier corrección.