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.

Sign up for GitHub

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

Jump to bottom

renderToNodeStream translation #646

Merged
merged 8 commits into from
Apr 2, 2023
Merged

renderToNodeStream translation #646

merged 8 commits into from
Apr 2, 2023

Conversation

cosmoart
Copy link
Contributor

@cosmoart cosmoart commented Mar 22, 2023 •
edited
Loading

#500 Traducción de renderToNodeStream

Me tome la libertad de traduccir "Node.js Readable Stream" como "Stream legible de Node.js."

@github-actions github-actions bot added the beta label Mar 22, 2023
@carburo carburo mentioned this pull request Mar 24, 2023
Open
carburo
carburo requested changes Mar 31, 2023
View reviewed changes
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! Te he dejando algunos comentarios y sugerencias para que revises antes de que mezclemos este PR.

beta/src/content/reference/react-dom/server/renderToNodeStream.md Outdated Show resolved Hide resolved
beta/src/content/reference/react-dom/server/renderToNodeStream.md

</Deprecated>

<Intro>

`renderToNodeStream` renders a React tree to a [Node.js Readable Stream.](https://nodejs.org/api/stream.html#readable-streams)
`renderToNodeStream` renderiza un árbol de React en un [*Stream* legible de Node.js.](https://nodejs.org/api/stream.html#readable-streams)
Copy link
Member

Choose a reason for hiding this comment

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

Me pregunto si deberíamos traducir «Readable» en «Readable Stream». Hasta el momento no he visto una traducción asentada y puede ser difícil para el lector saber de qué realmente se está hablando. ¿Qué criterio aplicaste para no traducir «Stream», pero sí «Readable»? Quisiera saber tu opinión en este caso, porque no tengo muy claro cuál sería la mejor opción.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Mis opciones de traducción fueron «Flujo legible de Node.js» y «Stream legible de Node.js». Elegí la segunda opción porque:

  • En varias publicaciones en español se usan ambos términos. Como en esta o esta.
  • En Node.js estoy más familiarizado con el termino "Stream" que "Flujo".

Copy link
Member

Choose a reason for hiding this comment

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

En varias publicaciones en español se usan ambos términos. Como en esta o esta.

Sí, el hecho de que no haya una consenso sobre qué término usar es el principal punto de mi comentario original
Seguramente será uno de los términos a revisitar luego de que se hayan hecho todas las traducciones para lograr una consistencia entre las diversas páginas de la documentación. De momento me parece bien y creo que podemos mezclar los cambios así.

beta/src/content/reference/react-dom/server/renderToNodeStream.md Outdated Show resolved Hide resolved
beta/src/content/reference/react-dom/server/renderToNodeStream.md Outdated Show resolved Hide resolved
@github-actions
Copy link

github-actions bot commented Mar 31, 2023 •
edited
Loading

Size Changes

📦 Next.js Bundle Analysis

This analysis was generated by the next.js bundle analysis action 🤖

This PR introduced no changes to the javascript bundle 🙌

@carburo carburo merged commit f003f1a into reactjs:main Apr 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@carburo carburo carburo approved these changes

Assignees
No one assigned
Labels
beta
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cosmoart @carburo