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

File and FileReader #325

Merged
merged 23 commits into from Aug 3, 2020
Merged

File and FileReader #325

merged 23 commits into from Aug 3, 2020

Conversation

daguitosama
Copy link
Contributor

Spanish translation.

Spanish translation
@CLAassistant
Copy link

CLAassistant commented Jul 26, 2020

CLA assistant check
All committers have signed the CLA.

correcciones
@vplentinax vplentinax changed the title Update article.md Fime and FileReader Jul 26, 2020
@vplentinax vplentinax changed the title Fime and FileReader File and FileReader Jul 26, 2020
@vplentinax
Copy link
Contributor

Hola! Siempre recuerda que el PR lleva el nombre del artículo traducido. Así el bot te lo asigna en el Issue 17.

Un saludo

4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

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

@joaquinelio
Copy link
Member

No lo revisé,
pero cuidemos que coincidadn los numeros de linea porq facilitan: revision y mantenim cdo cambia el ingles.

solo lei los suggest de @danilobrinu y por lo pronto estoy de acuerdo
*salvo 137 que dice URI o URl (jeje) en lugar de url ¿algun motivo para mayusscula?
y quizass traduccion de parametros dentro de codigo js, eso lo dejamos a eleccion hay q ser muy cuidadoso
*tambien sugiero poner explicacione a las sugerencias que no sesan obvias como los tildes,
ej "File aca no se traduce porq se refiere al nombre de la clase". La explicacion fuera de los marcadores
" ` " no se entrometen en el cambio sugerido.

@dbritto-dev
Copy link
Contributor

dbritto-dev commented Jul 28, 2020

No lo revisé,
pero cuidemos que coincidadn los numeros de linea porq facilitan: revision y mantenim cdo cambia el ingles.

solo lei los suggest de @danilobrinu y por lo pronto estoy de acuerdo
*salvo 137 que dice URI o URl (jeje) en lugar de url ¿algun motivo para mayusscula?
y quizass traduccion de parametros dentro de codigo js, eso lo dejamos a eleccion hay q ser muy cuidadoso
*tambien sugiero poner explicacione a las sugerencias que no sesan obvias como los tildes,
ej "File aca no se traduce porq se refiere al nombre de la clase". La explicacion fuera de los marcadores
" ` " no se entrometen en el cambio sugerido.

@joaquinelio La traducción de DATA URL es Datos URI, algunos pensarán que es enlace de datos o algo por el estilo pero lo que se implementa detrás hace que tenga sentido Datos URI debido a que dicha URI contiene el URI Schema.

@joaquinelio
Copy link
Member

@danilobrinu
Debo confesar que el tema ME SUPERA, ¡contento de tener un corrector qu refuerce la parte conceptual!
Pero en inglés también existe URI, (y URN que tampoico entendí)
¿debería cambiarse ahí también?
¿es importante, o un error? Si es asi podemos poner un PR al repo inglés... no mi prioridad claro.

@joaquinelio La traducción de DATA URL es Datos URI, algunos pensarán que es enlace de datos o algo por el estilo pero lo que se implementa detrás hace que tenga sentido Datos URI debido a que dicha URI contiene el URI Schema.

@dbritto-dev
Copy link
Contributor

dbritto-dev commented Jul 28, 2020

@danilobrinu
Debo confesar que el tema ME SUPERA, ¡contento de tener un corrector qu refuerce la parte conceptual!
Pero en inglés también existe URI, (y URN que tampoico entendí)
¿debería cambiarse ahí también?
¿es importante, o un error? Si es asi podemos poner un PR al repo inglés... no mi prioridad claro.

@joaquinelio La traducción de DATA URL es Datos URI, algunos pensarán que es enlace de datos o algo por el estilo pero lo que se implementa detrás hace que tenga sentido Datos URI debido a que dicha URI contiene el URI Schema.

Si justamente es por que cuando se habla de URI es debido a que dicha URL contiene un URI schema. En pocas palabras una URL con URI Schema es un URI.

sugested changes applied
@vplentinax
Copy link
Contributor

vplentinax commented Jul 31, 2020

@D4G0 , siempre que aceptes una sugerencia de los revisores y hagas nuevo commit, escribe un comentario que sólo diga: /done .
Así se detectan los nuevos cambios.

Un saludo!

Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Copy link
Contributor

@vplentinax vplentinax left a comment

Choose a reason for hiding this comment

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

Algunos pequeños cambios. También sugerí sobre dos sugerencias hechas previamente por @danilobrinu .

4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

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

@javascript-translate-bot

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

@vplentinax
Copy link
Contributor

@danilobrinu Disculpa, quería eliminar algunas revisiones que se superpusieron, y terminé por quitar algunas tuyas. Si no te es molestia, ¿puedes volver a hacer la revisión? Gracias y disculpa nuevamente.

Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
@dbritto-dev
Copy link
Contributor

@vplentinax creo que en este punto lo mejor seria que termines la revisión. Sino me equivoco ya están la mayoría de ellos.

daguitosama and others added 2 commits July 31, 2020 22:54
/done

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

Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
@vplentinax
Copy link
Contributor

@D4G0 Cuando termines de revisar y aceptar todos los cambios, escribe un comentario que solo diga: /done. Es necesario para que el bot nos avise para una nueva revisión o para aceptar la traducción. 😁

Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
@daguitosama
Copy link
Contributor Author

/done

4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
4-binary/04-file/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

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

Copy link
Contributor

@vplentinax vplentinax left a comment

Choose a reason for hiding this comment

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

Aprobado, añadiendo los nuevos cambios sugeridos por @danilobrinu

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.

No lo revisé ni lo lei ni se de qe se trata,
pero Bot me pide que apruebe y apruebo.
y confio en la revision que bot ignora, suerte tenemos un par de relevos
que el ruso puede considerar si caemos en la lucha...
EDIT:
Y metí el rabo donde no debía, ¿cómo es esto de 2 aprobaciones y "requiere cambios"?
Ah, yo también lo hacía, si los cambios que sugería no eran vitales pero ahora veo que es confuso...
Me atreví a dar a aceptar y corregir para terminar de una buena vez, las correcctiones eran espacios, algun gramatical y numeros de linea, @D4G0

joaquinelio and others added 8 commits August 3, 2020 13:24
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
Co-authored-by: Danilo Britto <ddbn.c2@gmail.com>
@joaquinelio joaquinelio merged commit 287a5dc into javascript-tutorial:master Aug 3, 2020
@javascript-translate-bot

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

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.

None yet

6 participants