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

Native prototypes #193

Merged
merged 2 commits into from Jul 3, 2020
Merged

Native prototypes #193

merged 2 commits into from Jul 3, 2020

Conversation

cortizg
Copy link
Contributor

@cortizg cortizg commented May 17, 2020

No description provided.

1-08-03-np Traducido 14

1-08-03-np Corregido 11

1-08-03-np Corregido 12
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.

1-8-3-2 solution, no lo entiendo que pasa , y no es una edicion fácil sin branch
PARECE que está traducido pero no borradas las lineas ingles y por eso tiene 20 lineas má´s

ojo esta vez no revisé demasiado

queja:
aparece el importantísimo wrapper de nuevo acá,
bellamente escondido detrás de una buena traducción.


```js run
let obj = {};
alert( obj ); // "[object Object]" ?
```

Where's the code that generates the string `"[object Object]"`? That's a built-in `toString` method, but where is it? The `obj` is empty!
¿Dónde está el código que genera la cadena `"[objeto Objeto]"`? Ese es un método integrado `toString`, pero ¿dónde está? ¡El `obj` está vacío!
Copy link
Member

Choose a reason for hiding this comment

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

¿Dónde está el código que genera la cadena "[objet Objet]"? Ese es un método integrado toString, pero ¿dónde está? ¡El obj está vacío!

Noes descripción, es el resultado del alert [objet Objet] supongo el 2do en mayussc porq es clase.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

lo cierro?

Copy link
Member

Choose a reason for hiding this comment

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

@cortizg
Hay una alternativa, la hice antes:
Hago MERGE, el bot es entusiasma y lo marca hecho,
lo desmarco y listo, sigue pendiente
creo que es más fácil.

Entonces vos podes hacer un nuevo PR dede ahi,
pero tendras que usar el repo EN como base, algo qye habrás hecho antes
y el reviewer también tendrá que contrastar con el repo EN en lugar de seguir los colorcitos lo que es ¡todo un drama! ja ja
¿merge y desmarco?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

¡probemos!!

Copy link
Member

Choose a reason for hiding this comment

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

¡probemos!!
@cortizg
probemos, nada. Ya lo hice.
El repo tiene ya tu traducción defectuosa 👍 Te corresponde arreglarlo ahora.
En el issue17 sigue asignado, el # borrado.
Tendr´as que abrir esos 5 archivos, modificarlos y nuevo pr. Le isse un hiziu para asignártelo pero no me dejó, así que te enteras por este medio comentario.

fede rastas
Comentario entero no medio. "...medio, comentario"

@javascript-translate-bot

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


```js run
function f() {}

alert(f.__proto__ == Function.prototype); // true
alert(f.__proto__.__proto__ == Object.prototype); // true, inherit from objects
alert(f.__proto__ == Function.prototype); // ferdadero
Copy link
Contributor Author

@cortizg cortizg Jul 2, 2020

Choose a reason for hiding this comment

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

// verdadero

@joaquinelio joaquinelio merged commit 08eaccd into javascript-tutorial:master Jul 3, 2020
@javascript-translate-bot

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

@cortizg
Copy link
Contributor Author

cortizg commented Jul 5, 2020

Listo!

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

Successfully merging this pull request may close these issues.

None yet

3 participants