The Wayback Machine - https://web.archive.org/web/20201224035803/https://github.com/javascript-tutorial/es.javascript.info/pull/252
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

Symbol type #252

Merged
merged 10 commits into from Jul 5, 2020
Merged

Symbol type #252

merged 10 commits into from Jul 5, 2020

Conversation

@vplentinax
Copy link
Contributor

@vplentinax vplentinax commented Jun 13, 2020

Este PR corresponde a la traducción del PR Symbol type #115 hecho por la usuaria @mariapb. Revisar.Dicho PR #115 será cerrado y editado en el Issue #17

@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-es Jun 13, 2020
@iliakan iliakan mentioned this pull request Jun 13, 2020
159 of 174 tasks complete
1-js/04-object-basics/08-symbol/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

Copy link
Contributor

@cortizg cortizg left a comment

Algunas observaciones

1-js/04-object-basics/08-symbol/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/08-symbol/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/08-symbol/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/08-symbol/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/08-symbol/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

1-js/04-object-basics/08-symbol/article.md Outdated Show resolved Hide resolved
name: "John"
};
let id = Symbol("id");
user[id] = 1;
alert( user[id] ); // we can access the data using the symbol as the key
alert( user[id] ); // podemos accesar a la información utilizando symbol como propiedad

This comment has been minimized.

@joaquinelio

joaquinelio Jun 14, 2020
Contributor

Suggested change
alert( user[id] ); // podemos accesar a la información utilizando symbol como propiedad
alert( user[id] ); // podemos accesar a la información utilizando el symbol como clave
1-js/04-object-basics/08-symbol/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

vplentinax and others added 3 commits Jun 16, 2020
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax
Copy link
Contributor Author

@vplentinax vplentinax commented Jun 16, 2020

/done

@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-es Jun 16, 2020
Copy link
Contributor

@joaquinelio joaquinelio left a comment

@vplentinax @ezzep66 @cortizg
Al toro por las... y seguir adelante.

En este particular artículo queda evidente que no va "key = property",
se confunde el nombre con la cosa o la llave con la caja, escribir propiedad/valor lleva a definición circular, son categorías diferentes y, ej: Mientras prop puede ser un boolean, key no.

En la mayoría de las referencias la distinción es superflua y puede pasar aunque no me guste, aunque acá la necesitamos.
Pero estaría mal usar "clave" si no se usa en el resto del curso. Así que refiguro
key (nombre de propiedad) / value (valor de propiedad)

Creo que el artículo quedó entendible. Si quieren cambiar, ok, me salgo de la discusión y sigo aportando con otras reviews. Que no nos frene.
FESTEJEMOS EL TERCIO DE REPO TRADUCIDO

NOTA: Quise añadir las sugerencias de c4 y no pude ¿como se hace suggest de una línea no traducida? Hay //comments no traducidos.

1-js/04-object-basics/08-symbol/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/08-symbol/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/08-symbol/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/08-symbol/article.md Outdated Show resolved Hide resolved

By specification, object property keys may be either of string type, or of symbol type. Not numbers, not booleans, only strings or symbols, these two types.
Por especificación, las claves de un objeto deben ser solamente del tipo String o Symbol. Solamente esos dos: String o Symbol.

This comment has been minimized.

@joaquinelio

joaquinelio Jul 2, 2020
Contributor

Suggested change
Por especificación, las claves de un objeto deben ser solamente del tipo String o Symbol. Solamente esos dos: String o Symbol.
Por especificación, los nombres de propiedad (keys) de un objeto deben ser solamente del tipo String o Symbol. Solamente esos dos: String o Symbol.
1-js/04-object-basics/08-symbol/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

vplentinax and others added 6 commits Jul 2, 2020
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
En el glosario se estipuló key = clave. Así lo dejo y dejaré en próximas traducciones.
@vplentinax
Copy link
Contributor Author

@vplentinax vplentinax commented Jul 2, 2020

/done

@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-es Jul 2, 2020
Copy link
Contributor

@joaquinelio joaquinelio left a comment

@vplentinax

En el glosario se estipuló key = clave. Así lo dejo y dejaré en próximas traducciones.

¡De acuerdo!
Pero... Algo me perdí, insistí con clave pero me había cansado... Recuerdo la argumentacipn de eze sobre propiedad/valor, del ctl+F y el cuenta palabbras, deque era como se lo enseñaron...
Y veo qye lo repusiste< ¡hace dos semanas!

clave, perfecto. Es la llave con la que se accede al value de la propiedad.

@vplentinax vplentinax merged commit d0b203c into javascript-tutorial:master Jul 5, 2020
1 check passed
1 check passed
license/cla Contributor License Agreement is signed.
Details
@javascript-translate-bot

Error: the article has another translator @mariabp, not @vplentinax in the Progress Issue #17. Please update the progress issue or ask maintainers to do so.

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

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.
HTTPS · web.archive.org
← Home