Conversation
carburo
requested changes
Mar 31, 2023
Member
carburo
left a comment
There was a problem hiding this comment.
¡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.
Contributor
Author
|
Gracias por la sugerencias. He arreglado los typos que señalaste y otros que se me pasaron la primera vez. |
carburo
approved these changes
Apr 6, 2023
Size ChangesDetails📦 Next.js Bundle AnalysisThis analysis was generated by the next.js bundle analysis action 🤖 This PR introduced no changes to the javascript bundle 🙌 |
Member
|
¡Gracias! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hola. Estoy atento a cualquier corrección.