Resumable file upload #375
Conversation
Primer día de trabajo.
|
|
There was a problem hiding this comment.
Gracias por tu contribución @FroggyGentlemen. Solo queda corregir unas faltas ortográficas que encontré para poder aprobarlo.
|
Please make the requested changes. After it, add a comment "/done". |
There was a problem hiding this comment.
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ó)
| xhr.send(file.slice(startByte)); | ||
| ``` | ||
|
|
||
| Here we send the server both file id as `X-File-Id`, so it knows which file we're uploading, and the starting byte as `X-Start-Byte`, so it knows we're not uploading it initially, but resuming. | ||
| Aqui enviamos al server ambos archivos id como `X-File-Id`, de modo que sepa que archivos estamos cargando, y el byte inicial como `X-Start-Byte`, de modo que sepa que no lo estamos cargando inicialmente, si no que reanudandolo. |
There was a problem hiding this comment.
¿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.
| Aqui enviamos al server ambos archivos id como `X-File-Id`, de modo que sepa que archivos estamos cargando, y el byte inicial como `X-Start-Byte`, de modo que sepa que no lo estamos cargando inicialmente, si no que reanudandolo. | |
| Aquí enviamos al servidor dos cosas: el id del archivo `X-File-Id`, para que sepa qué archivo estamos cargando, y el byte inicial `X-Start-Byte`, para que sepa que no estamos iniciando una carga sino reanudándola. |
|
|
||
| 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 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. |
There was a problem hiding this comment.
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 exactamente X-Start-Byte, agregarle los datos.
pero mejor más parecido al de uds, más claro.
| 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 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. | |
| El servidor debe verificar sus registros y, si estaba cargando ese archivo, y si el tamaño de esa carga es exactamente `X-Start-Byte`, entonces agregarle los datos. |
|
Please make the requested changes. After it, add a comment "/done". |
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
Co-authored-by: Valentina VP <34555644+vplentinax@users.noreply.github.com>
|
/done |
|
Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉 |
|
Emmm... yo no había revisado los cambios y tampoco aprobado....¿por qué se hizo merge? @EzequielCaste |
|
Es que ve mi nombre y se descompone No es nada, lo hizo de gaucho para evitarte el dolor de leerme. @vplentinax |
No description provided.