javascript-tutorial/es.javascript.infoPublic

NotificationsYou must be signed in to change notification settings
Fork230
Star601

Native prototypes#193

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

joaquinelio merged 2 commits intojavascript-tutorial:masterfromcortizg:es.javascript.info.1-08-03-np

Jul 3, 2020

Merged

Native prototypes#193

joaquinelio merged 2 commits intojavascript-tutorial:masterfromcortizg:es.javascript.info.1-08-03-np

Jul 3, 2020

Conversation

Copy link

Contributor

cortizg commentedMay 17, 2020•
edited
Loading

No description provided.

traduccion pre-final

5eeedec

1-08-03-np Traducido 141-08-03-np Corregido 111-08-03-np Corregido 12

javascript-translate-bot added the review needed label

May 17, 2020

javascript-translate-bot requested a review froma team

May 17, 2020 03:12

iliakan mentioned this pull request

May 17, 2020

Spanish Translation Progress#17

Open

Merge branch 'master' into es.javascript.info.1-08-03-np

fd20109

joaquinelio requested changes

Jul 1, 2020

View reviewed changes

Copy link

Member

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.

1-js/08-prototypes/03-native-prototypes/article.md


		```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 integradotoString, pero ¿dónde está? ¡Elobj está vacío!

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

Copy link

ContributorAuthor

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

ContributorAuthor

cortizgJul 4, 2020

Choose a reason for hiding this comment

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

¡probemos!!

Copy link

Member

joaquinelioJul 4, 2020•
edited
Loading

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 added changes requested and removed review needed labels

Jul 1, 2020

Copy link

javascript-translate-bot commentedJul 1, 2020

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

cortizg commented

Jul 2, 2020

View reviewed changes

1-js/08-prototypes/03-native-prototypes/article.md


		```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