GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Object.keys, values, entries by EzequielCaste · Pull Request #275 · javascript-tutorial/es.javascript.info

Conversation

@EzequielCaste

@EzequielCaste EzequielCaste commented Jun 22, 2020

Copy link
Copy Markdown
Contributor

ahora si el verdadero object keys

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@EzequielCaste
traducción object keys
af975d2
@EzequielCaste
EzequielCaste requested a review from vplentinax June 22, 2020 21:50
@javascript-translate-bot javascript-translate-bot added the review needed label Jun 22, 2020
@javascript-translate-bot
javascript-translate-bot requested a review from a team June 22, 2020 21:50
@iliakan iliakan mentioned this pull request Jun 22, 2020
Spanish Translation Progress #17
Open
joaquinelio
joaquinelio requested changes Jul 6, 2020
View reviewed changes

@joaquinelio joaquinelio left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

key
@ezzep66 @vplentinax
key no es propiedad:
La llave no es la caja. Ppropiedad es el nombre Y el valor asociado
.keys(obj) no devuelve propiedades, solo sus keys. (devuelve solo sus nombres)
" propiedades que utilizan Symbol como propiedades" definicipn circular.
Propiedad puede tener cualquier valor, KEY debe ser string porque es el nombre. (string o symbol).

En la vida real no hace falta la distinción, pero en este capítulo me parece importante,
Por lo menos, donde se hace evidente la contradicción, sugiero "nombre de propiedad" aunq cuando lo sugerí Valentina me reafirmó "establecimos que se traduce clave"
Y me escudo para no aprobarlo en request por sintaxis Pero que lo apruebe otro ¡ja ja!
pero es tu prerogativa y que esto no lo frene, prefiero avanzar. Merge si quiern , pero arreglen el tilde.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
Comment thread 1-js/05-data-types/09-keys-values-entries/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 1-js/05-data-types/09-keys-values-entries/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 1-js/05-data-types/09-keys-values-entries/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

Comment thread 1-js/05-data-types/09-keys-values-entries/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

@javascript-translate-bot javascript-translate-bot removed the review needed label Jul 6, 2020
@javascript-translate-bot

javascript-translate-bot commented Jul 6, 2020

Copy link
Copy Markdown

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

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@javascript-translate-bot javascript-translate-bot added the changes requested label Jul 6, 2020
@joaquinelio

joaquinelio commented Jul 6, 2020 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

@ezzep66
OTRA COSA
aparece " @vplentinax recentrly edited these files"
no encontré commit a "estos archivos"
supongo se refiere a los originales inglés, su último resync los alteró y en la review no los consideré

edit:
no problem, seguro que sale como conflicto 3way merge

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@EzequielCaste

EzequielCaste commented Jul 6, 2020

Copy link
Copy Markdown
Contributor Author

key
@ezzep66 @vplentinax
key no es propiedad:
La llave no es la caja. Ppropiedad es el nombre Y el valor asociado
.keys(obj) no devuelve propiedades, solo sus keys. (devuelve solo sus nombres)
" propiedades que utilizan Symbol como propiedades" definicipn circular.
Propiedad puede tener cualquier valor, KEY debe ser string porque es el nombre. (string o symbol).

En la vida real no hace falta la distinción, pero en este capítulo me parece importante,
Por lo menos, donde se hace evidente la contradicción, sugiero "nombre de propiedad" aunq cuando lo sugerí Valentina me reafirmó "establecimos que se traduce clave"
Y me escudo para no aprobarlo en request por sintaxis Pero que lo apruebe otro ¡ja ja!
pero es tu prerogativa y que esto no lo frene, prefiero avanzar. Merge si quiern , pero arreglen el tilde.

❓

me encantaría coincidir contigo pero CADA VEZ que busco en internet alguna de tus afirmaciones, encuentro una refutación:
.keys(obj) no devuelve propiedades

El método Object.keys() devuelve un array de las propiedades names de un objeto, en el mismo orden como se obtienen en un loop normal

Te pido por favor @joaquinelio si vas a afirmar algo por favor que sea basado en una fuente oficial como MDN, NO en tus propias conclusiones.

Fuente

Otra fuente

Google es tu amigo 👍

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

EzequielCaste and others added 3 commits July 6, 2020 11:37
@EzequielCaste @joaquinelio
Update 1-js/05-data-types/09-keys-values-entries/article.md
fe412c8
Co-authored-by: joaquinelio <[email protected]>
@EzequielCaste @joaquinelio
Update 1-js/05-data-types/09-keys-values-entries/article.md
82768d5
Co-authored-by: joaquinelio <[email protected]>
@EzequielCaste @joaquinelio
Update 1-js/05-data-types/09-keys-values-entries/article.md
7288343
Co-authored-by: joaquinelio <[email protected]>
EzequielCaste
EzequielCaste commented Jul 6, 2020
View reviewed changes
Comment thread 1-js/05-data-types/09-keys-values-entries/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

@EzequielCaste
Update 1-js/05-data-types/09-keys-values-entries/article.md
5dd44d3
EzequielCaste
EzequielCaste commented Jul 6, 2020
View reviewed changes
Comment thread 1-js/05-data-types/09-keys-values-entries/article.md Resolved

Uh oh!

There was an error while loading. Please reload this page.

@EzequielCaste
Update 1-js/05-data-types/09-keys-values-entries/article.md
9e60387
@joaquinelio

joaquinelio commented Jul 7, 2020 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

TLDR, sorry. Sabés que podés ignorarme y hacer el merge, este capítulo no va a hacer peores programadores.

key
@ezzep66 @vplentinax
key no es propiedad:
La llave no es la caja. Ppropiedad es el nombre Y el valor asociado
.keys(obj) no devuelve propiedades, solo sus keys. (devuelve solo sus nombres)
" propiedades que utilizan Symbol como propiedades" definicipn circular.
Propiedad puede tener cualquier valor, KEY debe ser string porque es el nombre. (string o symbol).
En la vida real no hace falta la distinción, pero en este capítulo me parece importante,
Por lo menos, donde se hace evidente la contradicción, sugiero "nombre de propiedad" aunq cuando lo sugerí Valentina me reafirmó "establecimos que se traduce clave"
Y me escudo para no aprobarlo en request por sintaxis Pero que lo apruebe otro ¡ja ja!
pero es tu prerogativa y que esto no lo frene, prefiero avanzar. Merge si quiern , pero arreglen el tilde.

❓

me encantaría coincidir contigo pero CADA VEZ que busco en internet alguna de tus afirmaciones, encuentro una refutación:
.keys(obj) no devuelve propiedades

El método Object.keys() devuelve un array de las propiedades names de un objeto, en el mismo orden como se obtienen en un loop normal

Te pido por favor @joaquinelio si vas a afirmar algo por favor que sea basado en una fuente oficial como MDN, NO en tus propias conclusiones.

Fuente

Otra fuente

Google es tu amigo 👍

Bueno, tus fuentes están equivocadas. ¡ja ja!
Googlees hoy mi único amigo... pero ¿no te dijeron que es medio bolacero?

Lo que pasa que esta distinción es innecesaria para el que programa, solo me quejo en este capítulo.
Es perfecto decir car["color"] y tratar el nombre "color" como si fuera la propiedad.
Pero "color" no dice nada si no lo ponés en la cerradura que lo asocia a value. El array es uno de strings.

Debí numerar los argumentos, cada renglon es uno. El que más molesta es la definicion circular, el que key solo puede ser (porque es solo el nombre) string (o symbol, uf) el más contundente.

Voy a usar el ej de tu fuente #otra
prop = "color" // string, es solo el nombre
if (key !== prop) // compara strings. ni key ni prop podrian ser otra cosa. (salvo symbol uf)
object[key] = car[key] // car["brand"]
La propiedad tiene un valor, "blue". pero "color" no.

MDN es a veces peor que nosotros, tiene varios "falsos amigos", pero muchos seguramente se arrastran de antes.
Quizás cuando terminemos me voy para ahí para hacer nuevos adversarios... https://wiki.developer.mozilla.org/es/docs/MDN/Contribute/Editor 🚀
Mentira, estoy ACÁ porque es tutorial, la puerta de entrada al mundo JS. Despues deberían estudiar inglés.

Personal, y no relacionado con key.
Decidí hace un año sumarme a la traducción por un motivo: javascript.info EN es mejor que MDN EN y googledev EN. El mejor golpazo lo tuve con un proyecto con localstorage, antes de que el ruso lo agregara.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@EzequielCaste
Merge branch 'master' into the-real-Object.keys
4773cac
@EzequielCaste
EzequielCaste removed the request for review from vplentinax July 11, 2020 15:16
EzequielCaste
EzequielCaste commented Jul 11, 2020
View reviewed changes

@EzequielCaste EzequielCaste left a comment

Copy link
Copy Markdown
Contributor Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

ok

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@EzequielCaste

EzequielCaste commented Jul 11, 2020

Copy link
Copy Markdown
Contributor Author

/done

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@javascript-translate-bot javascript-translate-bot added review needed and removed changes requested labels Jul 11, 2020
@javascript-translate-bot
javascript-translate-bot requested a review from a team July 11, 2020 15:17
EzequielCaste
EzequielCaste commented Jul 11, 2020
View reviewed changes

@EzequielCaste EzequielCaste left a comment

Copy link
Copy Markdown
Contributor Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

ok

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
EzequielCaste
EzequielCaste commented Jul 11, 2020
View reviewed changes

@EzequielCaste EzequielCaste left a comment

Copy link
Copy Markdown
Contributor Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

ok

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@EzequielCaste

EzequielCaste commented Jul 11, 2020

Copy link
Copy Markdown
Contributor Author

/done

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

EzequielCaste
EzequielCaste commented Jul 11, 2020
View reviewed changes

@EzequielCaste EzequielCaste left a comment

Copy link
Copy Markdown
Contributor Author

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

/done

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
joaquinelio
joaquinelio approved these changes Jul 12, 2020
View reviewed changes

@joaquinelio joaquinelio left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

ok

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@javascript-translate-bot javascript-translate-bot added needs +1 and removed review needed labels Jul 12, 2020
@vplentinax
vplentinax self-requested a review July 13, 2020 00:17
vplentinax
vplentinax approved these changes Jul 13, 2020
View reviewed changes

@vplentinax vplentinax left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Ok!

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@vplentinax vplentinax added ready to merge and removed needs +1 labels Jul 13, 2020
@joaquinelio
joaquinelio merged commit b46487d into javascript-tutorial:master Jul 13, 2020
@javascript-translate-bot

javascript-translate-bot commented Jul 13, 2020

Copy link
Copy Markdown

Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@joaquinelio joaquinelio joaquinelio approved these changes
@vplentinax vplentinax vplentinax approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

ready to merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

4 participants

@EzequielCaste @javascript-translate-bot @joaquinelio @vplentinax
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.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.