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

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

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

Choose a reason for hiding this comment

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

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
Copy link
Member

Choose a reason for hiding this comment

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

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 June 16, 2020 12:09
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

/done

Copy link
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

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

@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.
Copy link
Member

Choose a reason for hiding this comment

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

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 July 2, 2020 17:59
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

/done

Copy link
Member

@joaquinelio joaquinelio left a comment

Choose a reason for hiding this comment

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

@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
@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
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants