Skip to content

sync27Agosto #361

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

Merged
merged 5 commits into from
Aug 31, 2020
Merged

sync27Agosto #361

merged 5 commits into from
Aug 31, 2020

Conversation

vplentinax
Copy link
Contributor

Actualización desde el 25 de Julio hasta hoy.

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.

Ya entendí
Hay cosass que recordaba arregladas, es por el abandono de pr estos dias

algunas mods estan ya en los pr,
otras son pr mios y de maksumi al inglés

PREGUNTA: CUAL ES EL PROCEDIMIENTO QUE SEGUIS PARA LAS SYNC?

acaso comparas todo el repo?
miras solo los ultimos pr o commit del ingles?
primero me produce admiración, (traté de hacerlo con lens y con merge y es abrumador)
segundo porque pretendo que cambies el modus operandi... =)

@@ -1,6 +1,6 @@
# Operadores Lógicos

Hay tres operadores lógicos en JavaScript: `||` (OR (O)), `&&` (AND (Y)), `!` (NOT (NO)).
Hay tres operadores lógicos en JavaScript: `||` (O), `&&` (Y), `!` (NO).
Copy link
Member

Choose a reason for hiding this comment

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

No estoy seguro de que me guste quitar el nombre de las operaciones.

Desde que estudíe lógica electrónica en 1977 (o antes, soy autodidacta) que uso
AND OR NOT XOR NAND NOR para las operaciones a nivel de bit.

No lo pongo como suggest porque tampoco me gusta la oracion que parece jeroglifico. NADIE lo va a entender y se van a desanimar.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Por eso hice el cambio, por legibilidad.

Copy link
Member

Choose a reason for hiding this comment

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

👍

```warn header="Internet Explorer and Edge: no support for `x/y`"
Internet Explorer and Edge don't support `x/y` properties for historical reasons.
```warn header="Internet Explorer: no support for `x/y`"
Internet Explorer doesn't support `x/y` properties for historical reasons.
Copy link
Member

Choose a reason for hiding this comment

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

Acá hay un problema.
RECUERDO que esto estaba arreglado, warn Edge x/y LO VI... ¿o era otro lado?
lo chequo en un rato.

@@ -15,7 +15,7 @@ Also, popups are tricky on mobile devices, that don't show multiple windows simu

Still, there are tasks where popups are still used, e.g. for OAuth authorization (login with Google/Facebook/...), because:

1. A popup is a separate window with its own independent JavaScript environment. So opening a popup with a third-party non-trusted site is safe.
1. A popup is a separate window with its own independent JavaScript environment. So opening a popup from a third-party non-trusted site is safe.
Copy link
Member

Choose a reason for hiding this comment

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

Ya entendi,
Esta linea no tenia sentido en ingles,
mande pr, Ilya lo rechazó pero modificó lo importante para que no sea absurda.
La linea en español está mucho mas clara... espero que con el conflict no la desshagan.

@javascript-translate-bot

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

vplentinax and others added 4 commits August 30, 2020 15:22
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
@vplentinax
Copy link
Contributor Author

Ya entendí
Hay cosass que recordaba arregladas, es por el abandono de pr estos dias

algunas mods estan ya en los pr,
otras son pr mios y de maksumi al inglés

PREGUNTA: CUAL ES EL PROCEDIMIENTO QUE SEGUIS PARA LAS SYNC?

acaso comparas todo el repo?
miras solo los ultimos pr o commit del ingles?
primero me produce admiración, (traté de hacerlo con lens y con merge y es abrumador)
segundo porque pretendo que cambies el modus operandi... =)

últimos commits para sincronización
artículo por artículo para revisión de traducciones.

Si tu cambio no me aporta más rápidez dudo que lo acepte jajajaja. Me gusta mi forma.

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.

👍

Nota:
Me dio cierto pudor eso de "la secretaria que acepta llamadas y molesta al jefe", me dan miedo las feministas...

No me habia percatado que una linea es la misma que eze me esta peleando en otro pr...

Sobre el "modus operandi" del sync, hay motivo.
Luego me explayo (tengo 400 archivos con conflicto, revise/cambie 300 faltan 100... mas los conflictos de estass dos semanas... Sabiendo que quizas deba tirar todo porque sos mucho más eficaz)

@joaquinelio joaquinelio merged commit 94f389f into javascript-tutorial:master Aug 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants