javascript-tutorial/es.javascript.infoPublic

NotificationsYou must be signed in to change notification settings
Fork230
Star600

Resumable file upload#375

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

EzequielCaste merged 24 commits intojavascript-tutorial:masterfromFroggyGentlemen:master

Sep 18, 2020

Merged

Resumable file upload#375

EzequielCaste merged 24 commits intojavascript-tutorial:masterfromFroggyGentlemen:master

Sep 18, 2020

Conversation

Copy link

Contributor

FroggyGentlemen commentedSep 7, 2020

No description provided.

FroggyGentlemen added2 commits

August 27, 2020 18:10

Update article.md

cb35c9f

Primer día de trabajo.

Update article.md

fd0d199

javascript-translate-bot added the review needed label

Sep 7, 2020

Copy link

CLAassistant commentedSep 7, 2020•
edited
Loading

All committers have signed the CLA.

vplentinax requested changes

Sep 8, 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.

Gracias por tu contribución@FroggyGentlemen. Solo queda corregir unas faltas ortográficas que encontré para poder aprobarlo.

5-network/09-resume-upload/article.md OutdatedShow resolvedHide resolved

javascript-translate-bot added changes requested and removed review needed labels

Sep 8, 2020

Copy link

javascript-translate-bot commentedSep 8, 2020

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

vplentinax requested a review fromjoaquinelio

September 8, 2020 02:13

iliakan mentioned this pull request

Sep 8, 2020

Spanish Translation Progress#17

Open

joaquinelio requested changes

Sep 8, 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.

Bienvenido

re-revisé-rescribí un par de lineas que me parecen mal expresadas en el original .

La primera vez que reviso con una revisión en curso, no sé si es efectivo.

@vplentinax , espero no te moleste edité 2 comentss tuyos (así no te robo crédito, ja)
("cuál" y "reanudar" en lugar de "resumir" que se te escapó)

5-network/09-resume-upload/article.md OutdatedShow resolvedHide resolved

5-network/09-resume-upload/article.md Outdated

		xhr.send(file.slice(startByte));
		```

		Here we send theserverboth file idas `X-File-Id`,so it knows which file we're uploading, and the startingbyteas`X-Start-Byte`,so it knows we're not uploading it initially, but resuming.
		Aqui enviamos alserverambos archivos idcomo `X-File-Id`,de modo que sepa que archivos estamos cargando, y elbyteinicial como`X-Start-Byte`,de modo que sepa que no lo estamos cargando inicialmente, si no que reanudandolo.

Copy link

Member

joaquinelioSep 8, 2020

Choose a reason for hiding this comment

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

¿A quién queres más, a mamá o a papá? 😆

Corrijo desde la correcta corrección de Valentina.

Tampoco está mal "de modo", creo que está poniendo algo más universal.

Para no estar tan pegados al inglés "archivo id", "id del archivo"

Mmm. Tampoco me gusta el original inglés "initially"

Mmm. Se malinterpreta quiénes son "ambos": file-id, start-byte. No son "ambos archivos"
En inglés está bien pero es confuso, en español no podemos estructurarlo así,
podemos poner una coma despues de ambos, prefiero dos puntos.

Acentoss: acá estan ambas versiones de "que":
qué estamos cargando (qué archivo)
que estamos cargando (que sepa eso)

"sino", el que opone "no ese sino otro"

y las comas separando estructuras

y qué tanto floreo, es confuso. Fuera.

Debería mandar PR al inglés, el original es una porquería
Seguramente la rechaza (no soy tan bueno) pero también seguramente entiende y la reescribe

Uf, edité esto varias veces...

AHORA se entiende.

Suggested change

	Aqui enviamos alserver ambos archivosidcomo`X-File-Id`,de modoque sepaque archivos estamos cargando, y el byte inicialcomo`X-Start-Byte`,de modoque sepa que noloestamoscargando inicialmente, si no que reanudandolo.
	Aquí enviamos alservidor dos cosas: eliddel archivo`X-File-Id`,paraque sepaqué archivo estamos cargando, y el byte inicial `X-Start-Byte`,paraque sepa que no estamosiniciando una carga sino reanudándola.

5-network/09-resume-upload/article.md Outdated


		The server should check its records, and if there was an upload of that file, and the current uploaded size is exactly `X-Start-Byte`, then append the data to it.
		El server deberia verificar sus registros, y en el caso de haber una carga de ese archivo, y el tamaño actual de la carga es exactamenteThe server should check its records, and if there was an upload of that file, and the current uploaded size is exactly `X-Start-Byte`, then append the data to it.

Copy link

Member

joaquinelioSep 8, 2020

Choose a reason for hiding this comment

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

Aquí te dejo otro dilema.

Pero quiero que no te preocupes y sepas que a pesar de nuestras diferencias con Valentina nos seguimos queriendo.
jaja

Pero no está mal (el anterior sí) es solo una sugerencia, ignorenla si les parece rebuscado.

como purista,
El servidor debe verificar sus registros y, si estaba cargando ese archivo y el tamaño de esa carga es exactamenteX-Start-Byte, agregarle los datos.

pero mejor más parecido al de uds, más claro.

Suggested change

	Elserver deberia verificar sus registros, y en el caso de haber una carga deese archivo, y el tamañoactualdela carga es exactamente The server should check its records, and if there was an upload of that file, and the current uploaded size is exactly`X-Start-Byte`,then append the data to it.
	Elservidor debe verificar sus registros y, si estaba cargandoese archivo, ysiel tamaño deesa carga es exactamente `X-Start-Byte`,entonces agregarle los datos.

Copy link

javascript-translate-bot commentedSep 8, 2020

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

FroggyGentlemenand others added18 commits

September 18, 2020 16:48