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

added translation for state a components memory #586

Merged

Conversation

samarjaffal
Copy link
Contributor

Copy link
Contributor

@Vicenciomf1 Vicenciomf1 left a comment

Choose a reason for hiding this comment

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

Hola @samarjaffal! Muchas gracias por tu aporte! Agregué y cambié algunas cosas para mantener el estilo de escritura de los capítulos anteriores de esta documentación y de la anterior, además de algunas cosas frecuentes que suele corregir @carburo, así podemos agilizar el merge de tu PR.
Pd: Vi tu presentación en la miduconf y fue muy inspiracional, gracias por los consejos.

beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
@github-actions
Copy link

github-actions bot commented Oct 29, 2022

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 🙌

beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
beta/src/content/learn/state-a-components-memory.md Outdated Show resolved Hide resolved
…into translation-for-state-a-components-memory
@samarjaffal
Copy link
Contributor Author

Hola @Vicenciomf1
¡Muchas gracias por las sugerencias y correcciones!
Voy a estar actualizando mi rama con las correcciones y tan pronto las tenga listas, les aviso.

Y muchas gracias por ver mi charla, significa mucho para mí 🤗

@samarjaffal
Copy link
Contributor Author

@Vicenciomf1 Creo que ya hice todas las sugerencias que me colocaste. Estoy a la espera de saber si seguiremos usando la palabra "setear". Espero no se me haya pasado nada :)

@Vicenciomf1
Copy link
Contributor

@Vicenciomf1 Creo que ya hice todas las sugerencias que me colocaste. Estoy a la espera de saber si seguiremos usando la palabra "setear". Espero no se me haya pasado nada :)

Hola @samarjaffal! Revisé en MDN y en diversas ocasiones se ocupa tal término, por lo que no sería algún caso aislado que tú o yo hayamos interiorizado tal término en vez de establecedor o asignador, con respecto a la documentación anterior, se evita la traducción y sólo se le llama función, si @carburo está de acuerdo, puedo modificar las veces en las que se utilizó "asignadora" en el capítulo de las refs y agregarlo al glosario en la guía de estilos como "setter" a tal función, y "asignar" a "set", sería sólo eso y hacemos el merge.

Pd: Agregué unas cositas mínimas, pero está todo perfecto! Muchas gracias por tu aporte

Pd2: No hay nada que agradecer, cuando se hacen las cosas de tal manera, lo bueno llega solo (a mí parecer), y efectivamente haces "un impacto en la vida de los demás", además es un orgullo el que alguien hable de temas que mejoran la salud mental y el ambiente laboral, casi siempre la gente se enfoca sólo en lo técnico.

@carburo
Copy link
Member

carburo commented Oct 31, 2022

@Vicenciomf1
Creo que podemos postergar la decisión sobre esa terminología para no bloquear el PR. Podríamos crear posteriormente un hilo de debate para analizarlo. Después de todo la documentación aún está en beta y podemos aplicar luego lo que se decida a las páginas que ya están traducidas. Es importante mantener consistente la terminología en este caso porque en la nueva documentación a menudo se hace la distinción entre setting the state vs. updating the state y esa misma distinción debe quedar clara en las traducciones.

Copy link
Contributor

@Vicenciomf1 Vicenciomf1 left a comment

Choose a reason for hiding this comment

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

¡Listo! Lo tendré anotado entonces para que sea más fácil hacer las modificaciones en el futuro cuando se llegue a un consenso acerca del término :)

@Vicenciomf1
Copy link
Contributor

¡Gracias @samarjaffal y @carburo!

@Vicenciomf1 Vicenciomf1 merged commit 6b6c855 into reactjs:main Oct 31, 2022
@samarjaffal
Copy link
Contributor Author

Gracias a ustedes 🤗 @Vicenciomf1 @carburo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants