use trampa para trap, algo incomodo, pero es la trampa de osos que atrapa operaciones, ¿que otra cosa podia poner?
podia dejar TRAP, todo el mundo la entiende
El articulo original es excelente, pero deberia mejorar la granatuca y hacerla mas suave
Fui muy liberal en la traduccion no técnica
tambien extendi algunas cosas
Linea 966, le corte un pedazo porque estaba mal.
se podia arreglar dejandola y agregando algo como "no en el caso de ejemplo porque...",
o agragando una linea al codigo de ejemplo.
cuando lo resuelva Ilya, lo arreglamos aca, sera un sync como tantos

Proxy and Reflect

addaf6c

javascript-translate-bot added the review needed label

Nov 27, 2020

javascript-translate-bot requested a review froma team

November 27, 2020 03:37

Copy link

javascript-translate-bot commentedNov 27, 2020

Error: the article has another translator@maksumi in the Progress Issue#17.

Copy link

MemberAuthor

joaquinelio commentedNov 27, 2020•
edited
Loading

I know, Mr Bot.
But I dont care.
I stoled it.

wtf I've stolen it

joaquinelio mentioned this pull request

Nov 29, 2020

Spanish Translation Progress#17

Open

Copy link

Contributor

vplentinax commentedDec 13, 2020

Me lo pienso para comenzar esta revisión... Pronto lo haré

Copy link

MemberAuthor

joaquinelio commentedDec 14, 2020

Me lo pienso para comenzar esta revisión... Pronto lo haré

podes hacerla en comodas cuotas...

pero si podes dar algo de tiempo, la sync es siempre prioridad
Y las mantengo pequeñas para minimizar conflicto, no deberia llevar mucho tiempo.

vplentinax requested changes

Dec 18, 2020

View reviewed changes

Copy link

Contributor

vplentinax left a comment

Choose a reason for hiding this comment

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

Revision de tareas y del articulo hasta la línea 200.

1-js/99-js-misc/01-proxy/03-observable/solution.md OutdatedShow resolvedHide resolved

1-js/99-js-misc/01-proxy/article.md OutdatedShow resolvedHide resolved

javascript-translate-bot removed the review needed label

Dec 18, 2020

Copy link

javascript-translate-bot commentedDec 18, 2020

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

javascript-translate-bot added the changes requested label

Dec 18, 2020

Apply suggestions from@vplentinax

ea422f6

Co-authored-by: ᐯᑭᒪEᑎTIᑎᗩ᙭ ᐯᑭ <34555644+vplentinax@users.noreply.github.com>

Copy link

MemberAuthor

joaquinelio commentedDec 19, 2020•
edited
Loading

@vplentinax

No comprendo muy bien la frase, ¿podría mejorarse?

` 181
// casi original
El proxy debe reemplazar totalmente el objeto target en todo lugar. Nadie debe jamás hacer referencia al objeto target después de haberlo hecho el proxy. De otro modo sería fácil manipularlo.

    //¿puedo apartarme del original y repasar lo que es la envoltura (wrapper) ?

// 1ra verion sobreexplicada
El proxy debe reemplazar completamente al objeto "target" que envolvió . El proxy debe supervisar o reemplazar todas sus operaciones: nadie debe jamás hacer referencia al objeto target saltando tal envoltura. De otro modo sería fácil desbaratarlo.

// creo que quedo bien simplificado y contundente
El proxy debe reemplazar completamente al objeto "target" que envolvió: nadie debe jamás hacer referencia al objeto target saltando tal envoltura. De otro modo sería fácil desbaratarlo.

`
¿ s entiende ?
tu dirás...

TLDR
si otro programador usa "target" sin el proxy, el estado del proxy puede romperse, a eso se refiere con
klo que aca me comí " y en todo lugar", ligado a "nadie debe acceder", ¿puedo considerar que se sobrentiende?
me parecio mas claro sin aclarar...

reformulacion 181 "proxy vs target"

67c1298

vplentinax requested changes

Dec 23, 2020

View reviewed changes

Copy link

Contributor

vplentinax left a comment

Choose a reason for hiding this comment

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

Revisón hasta la línea 592. Ya vamos por la mitad. Realmente está muy bien, pocas cosas que arreglar.

1-js/99-js-misc/01-proxy/article.md OutdatedShow resolvedHide resolved

Copy link

javascript-translate-bot commentedDec 23, 2020

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

vplentinax reviewed

Dec 23, 2020

View reviewed changes

Copy link

Contributor

vplentinax left a comment•
edited
Loading

Choose a reason for hiding this comment

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

Me gustó, me parece más claro. Me refiero a lo que arreglaste en reformulacion 181 "proxy vs target"@joaquinelio

joaquinelioand others added2 commits

December 23, 2020 20:48

Apply suggestions II from@vplentinax

ad6aa67

Co-authored-by: ᐯᑭᒪEᑎTIᑎᗩ᙭ ᐯᑭ <34555644+vplentinax@users.noreply.github.com>

Legibilidad linea 328

b6f76cb

Copy link

MemberAuthor

joaquinelio commentedDec 23, 2020

@vplentinax
Reescribí linea 328 que seguia confusa.

vplentinax approved these changes

Jan 17, 2021

View reviewed changes

Copy link

Contributor

vplentinax left a comment

Choose a reason for hiding this comment

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

Pequeñas cositas, una vez aceptadas puedes hacer merge!

1-js/99-js-misc/01-proxy/article.md OutdatedShow resolvedHide resolved

1-js/99-js-misc/01-proxy/article.md Outdated


		For example,`getName()`method accesses the private`#name`property and breaks after proxying:
		Por ejemplo, el método`getName()`accede a la propiedad privada`#name`y falla por hacerlo proxy:

Copy link

Contributor

vplentinaxJan 17, 2021

Choose a reason for hiding this comment

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

Suggested change

	Por ejemplo, el método`getName()` accede a la propiedad privada`#name` yfalla por hacerlo proxy:
	Por ejemplo, el método`getName()` accede a la propiedad privada`#name` y se interrumpe después del proxy:

Copy link

MemberAuthor

joaquinelioJan 17, 2021

Choose a reason for hiding this comment

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

No, no se interrumpe, falla, porque no encuentra la variable.

Y no "después del proxy" en todo caso sería "después de convertirlo a proxy"
pero puse "POR" porque la falla es consecuencia de convertirlo.
Falla, por haberlo convertido en proxy.

No enconctré forma comoda de traducir el ubicuo "proxing" que traduje como "hacerlo proxy"
¡ proxificar ?

Copy link

MemberAuthor

joaquinelioJan 17, 2021

Choose a reason for hiding this comment

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

un problema que no vi de poner "hacerlo proxy" es q parece que convirtiera al metodo,
deberia aclarar que convierto el objeto entero o escribirlo mas vagamente

"...falla despues de hacer el proxy"
"...falla despues de la conversión a proxy"
"...falla depués de la proxificación. "

no me parece nada mal la idea. Es muy comodo tener la palabra proxificar.

tamb puedo sobreaclarar
"...falla depués de la proxificación del objeto user. "

@vplentinax , pienso poner
"...falla depués de la proxificación. "
¿que tal?

1-js/99-js-misc/01-proxy/article.md OutdatedShow resolvedHide resolved

javascript-translate-bot added needs +1 and removed changes requested labels

Jan 17, 2021

joaquinelioand others added5 commits

January 17, 2021 16:59

Update 1-js/99-js-misc/01-proxy/article.md

53c5dde

Co-authored-by: Valentiina VP <34555644+vplentinax@users.noreply.github.com>

Update 1-js/99-js-misc/01-proxy/article.md

d82a677

Co-authored-by: Valentiina VP <34555644+vplentinax@users.noreply.github.com>

Update 1-js/99-js-misc/01-proxy/article.md

06f2d20

Co-authored-by: Valentiina VP <34555644+vplentinax@users.noreply.github.com>

Merge branch 'master' into leproso

eec9603

Update article.md

20575b7

Copy link

MemberAuthor

joaquinelio commentedJan 18, 2021

Novedades

-Acepté (alguno de mala gana :P ) tus cambios.
-Menos uno que rechazo porque le erra (o yerra) al concepto, pero deje comentada una versión alternativa que creo mejor.
-También hubo 3 conflictos que acabo de resolver y podrias revisar. La mejora de Ilya es buena, mejor la explicacion y alternativas de manejo del metodo "revoke". lineas 966-972 (o podes cliquear los commits para ver las idas y venidas...)

joaquinelio added2 commits

January 24, 2021 23:24

Update article.md

fddf3b0

fix link a mdn proxy no func c/ [ ](mdn:)

5d16607

Copy link

MemberAuthor

joaquinelio commentedJan 25, 2021

Acepte todas las sug excepto linea 845 porque era un error conceptual que no puedo aceptar.

Es el unico articulo que falta de la seccion Js,
es el más largo de todo el repo,
el PR ya cumplio dos meses,

y la demora genera conflictos que tengo que andar resolviendo

El art es bueno, vale la pena el merge

joaquinelio merged commitedf7335 intojavascript-tutorial:master

Jan 25, 2021

Copy link

javascript-translate-bot commentedJan 25, 2021

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

joaquinelio deleted the leproso branch

January 25, 2021 02:48

Labels

needs +1

3 participants

Movatterモバイル変換

Proxy and Reflect#451

Proxy and Reflect#451

Uh oh!

Conversation

joaquinelio commentedNov 27, 2020• editedLoading Uh oh!There was an error while loading.Please reload this page.

Uh oh!

Uh oh!

javascript-translate-bot commentedNov 27, 2020

Uh oh!

joaquinelio commentedNov 27, 2020• editedLoading Uh oh!There was an error while loading.Please reload this page.

Uh oh!

Uh oh!

vplentinax commentedDec 13, 2020

Uh oh!

joaquinelio commentedDec 14, 2020

Uh oh!

vplentinax left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

javascript-translate-bot commentedDec 18, 2020

Uh oh!

joaquinelio commentedDec 19, 2020• editedLoading Uh oh!There was an error while loading.Please reload this page.

Uh oh!

Uh oh!

vplentinax left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

javascript-translate-bot commentedDec 23, 2020

Uh oh!

vplentinax left a comment• editedLoading Uh oh!There was an error while loading.Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

joaquinelio commentedDec 23, 2020

Uh oh!

vplentinax left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

vplentinaxJan 17, 2021

Choose a reason for hiding this comment

Uh oh!

joaquinelioJan 17, 2021

Choose a reason for hiding this comment

Uh oh!

joaquinelioJan 17, 2021

Choose a reason for hiding this comment

Uh oh!

Uh oh!

joaquinelio commentedJan 18, 2021

Uh oh!

joaquinelio commentedJan 25, 2021

Uh oh!

javascript-translate-bot commentedJan 25, 2021

Uh oh!

Uh oh!

joaquinelio commentedNov 27, 2020•
edited
Loading

joaquinelio commentedNov 27, 2020•
edited
Loading

joaquinelio commentedDec 19, 2020•
edited
Loading

vplentinax left a comment•
edited
Loading