Skip to content

/ es.javascript.info

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

Jump to bottom

Proxy and Reflect #451

Open
wants to merge 5 commits into
base: master
from
Open

Proxy and Reflect #451

wants to merge 5 commits into from
+328 −328

Conversation

@joaquinelio
Copy link
Contributor

@joaquinelio joaquinelio commented Nov 27, 2020 •
edited

Finally!
1033 lineas
ok, la mitad vacias, la mitad de mitad codigo
pero es el mas largo y complejo del repo

  1. 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

  2. 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

  3. 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
@joaquinelio
Proxy and Reflect
Loading status checks…
addaf6c
@javascript-translate-bot javascript-translate-bot requested a review from javascript-tutorial/translate-es Nov 27, 2020
@javascript-translate-bot
Copy link

@javascript-translate-bot javascript-translate-bot commented Nov 27, 2020

Error: the article has another translator @maksumi in the Progress Issue #17.
@joaquinelio
Copy link
Contributor Author

@joaquinelio joaquinelio commented Nov 27, 2020 •
edited

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

wtf I've stolen it
@joaquinelio joaquinelio mentioned this pull request Nov 29, 2020
Open
159 of 174 tasks complete
@vplentinax
Copy link
Contributor

@vplentinax vplentinax commented Dec 13, 2020

Me lo pienso para comenzar esta revisión... Pronto lo haré
@joaquinelio
Copy link
Contributor Author

@joaquinelio joaquinelio commented Dec 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
vplentinax requested changes Dec 18, 2020
View changes
Copy link
Contributor

@vplentinax vplentinax left a comment

Revision de tareas y del articulo hasta la línea 200.
1-js/99-js-misc/01-proxy/03-observable/solution.md Outdated Show resolved Hide resolved
1-js/99-js-misc/01-proxy/03-observable/solution.md Outdated Show resolved Hide resolved
1-js/99-js-misc/01-proxy/article.md Outdated Show resolved Hide resolved
1-js/99-js-misc/01-proxy/article.md Outdated Show resolved Hide resolved
1-js/99-js-misc/01-proxy/article.md Outdated Show resolved Hide resolved
1-js/99-js-misc/01-proxy/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot
Copy link

@javascript-translate-bot javascript-translate-bot commented Dec 18, 2020

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻
@joaquinelio @vplentinax
Apply suggestions from @vplentinax
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
ea422f6 
Co-authored-by: ᐯᑭᒪEᑎTIᑎᗩ᙭ ᐯᑭ <34555644+vplentinax@users.noreply.github.com>
@joaquinelio
Copy link
Contributor Author

@joaquinelio joaquinelio commented Dec 19, 2020 •
edited

@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...
@joaquinelio
reformulacion 181 "proxy vs target"
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
67c1298
@vplentinax
vplentinax requested changes Dec 23, 2020
View changes
Copy link
Contributor

@vplentinax vplentinax left a comment

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 Outdated Show resolved Hide resolved
1-js/99-js-misc/01-proxy/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot
Copy link

@javascript-translate-bot javascript-translate-bot commented Dec 23, 2020

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻
@vplentinax
vplentinax reviewed Dec 23, 2020
View changes
Copy link
Contributor

@vplentinax vplentinax left a comment •
edited

Me gustó, me parece más claro. Me refiero a lo que arreglaste en reformulacion 181 "proxy vs target" @joaquinelio
joaquinelio and others added 2 commits Dec 23, 2020
@joaquinelio @vplentinax
Apply suggestions II from @vplentinax
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
ad6aa67 
Co-authored-by: ᐯᑭᒪEᑎTIᑎᗩ᙭ ᐯᑭ <34555644+vplentinax@users.noreply.github.com>
@joaquinelio
Legibilidad linea 328
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
b6f76cb
@joaquinelio
Copy link
Contributor Author

@joaquinelio joaquinelio commented Dec 23, 2020

@vplentinax
Reescribí linea 328 que seguia confusa.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vplentinax vplentinax

Assignees
No one assigned
Labels
changes requested
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@joaquinelio @javascript-translate-bot @vplentinax
You can’t perform that action at this time.