URL objects - #376

Agus-c · 2020-09-07T22:43:16Z

hola! tuve algunas dudas con respecto a la traducción de algunas lineas (16, 17 por ej.), aunque intenté hacerlo de todas formas. lo siento, espero que no sea mucha molestia.
Ademas, vi algunas cosas raras en el original, como en la linea 65. Pero no estoy seguro de que sean errores o no (tampoco sabría como solucionarlo), así que decidí dejarlo como estaba.

ey

Primer dia de trabajo. Completo, aunque necesita revision, hay cosas con las que no estoy seguro

CLAassistant · 2020-09-07T22:43:24Z

All committers have signed the CLA.

vplentinax

Hola @Agus-c , entiendo tus dudas con las líneas 16 y 17, son bastante confusas, pero en la línea 17 el "set" se refería al mismo "base" al que se estaba describiendo. Es decir, si se establece (base) y aparte de eso la "url" es una ruta única, entonces sucede lo que dice el resto de la línea. Realmente resultaba confuso, tardé un ratito en tomarle el sentido.

Por todo lo demás, sólo unas correcciones ortográficas para que pueda aprobarlo. ¡Un saludo!

javascript-translate-bot · 2020-09-08T02:43:10Z

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

vplentinax

Disculpa, olvidé estas correcciones...

javascript-translate-bot · 2020-09-08T02:50:59Z

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

joaquinelio

Bienvenido.

Algun cambio.

SSolemos traducir linkss, pero con MDN :
Tiene traducciones incompletas,
Es muy visible el combo de cambiar idioma
El ruso lo está cambiando por un sistema de path relativo, no me voy a poner a cambiarlo

@vplentinax
No sé si traducir path, quizas deberia etar entre la opcionales

joaquinelio · 2020-09-08T10:23:41Z


- **`url`** -- the full URL or only path (if base is set, see below),
- **`base`** -- an optional base URL: if set and `url` argument has only path, then the URL is generated relative to `base`.
+- **`url`** -- La URL completa o único path (si base está establecido, mira debajo),


Uh, otro caso de "a quien queres mas"... jajaj

Está confuso el inglés así que no lo respeto.
¡Es mas facil! Los hubiera separado y no explicar tanto.
pones (url) o (path, urlbase)
Pero es más rebuscado que una concatenacion

edit: sí los separó despues, no dije nada.
pero la linea ess confusa, y con la coma agregada se entiende enseguida.

Además "unico" añade confusion, prefiero "unicamente"

Suggested change

- **`url`** -- La URL completa o único path (si base está establecido, mira debajo),

- **`url`** -- La URL completa, o solamente la ruta si se establece base (mira debajo),

joaquinelio · 2020-09-08T11:06:19Z

+## Codificación

-There's a standard [RFC3986](https://tools.ietf.org/html/rfc3986) that defines which characters are allowed in URLs and which are not.
+Existe un estándar [RFC3986](https://tools.ietf.org/html/rfc3986) que define cuales caracteres son permitidos en URLs y cuáles no.


pron. indef. m. y f. En la construcción cuál(es)… cuál(es)…, equivale a uno(s)… otro(s)… A cuál cubre, a cuál ciega, a cuál embiste.

como pronombre no lleva acento solo si es relativo "los cuales"

Suggested change

Existe un estándar [RFC3986](https://tools.ietf.org/html/rfc3986) que define cuales caracteres son permitidos en URLs y cuáles no.

Existe un estándar [RFC3986](https://tools.ietf.org/html/rfc3986) que define cuáles caracteres son permitidos en URLs y cuáles no.

javascript-translate-bot · 2020-09-08T11:32:34Z

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

joaquinelio · 2020-09-08T13:03:22Z

Una review sin esperar la anterior se ve confuso, sorry no lo vuelvo a hacer.

Sobre las cosas raras, es el markdown, .md es la extensión de markdown, como los tags htm para titulos.
El git tiene uno, pero el server del repo tiene otro más evolucionado asi que el preview aca se ve distinto al online.
Podess instalar sserver local para probar formato, links , y ejercicios

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: joaquinelio <joaquinelio@gmail.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: joaquinelio <joaquinelio@gmail.com>

Agus-c · 2020-09-17T20:29:14Z

/done

joaquinelio

ok.
pero...

alguna duda con el acento a cuales ? por ahi lo explique mal...
ejemplo de la real academia:
"Le explicó cuáles eran sus razones." linea 119 va acentuada
(como pron relativo sin acento es "le explicó sus razones, las cuales...")
aprobado y que no se pierda mas tiempo, pero la suggest stands.

vplentinax

Ok

javascript-translate-bot · 2020-09-20T19:15:55Z

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

Agus-c added 3 commits August 25, 2020 19:16

Update article.md

898fe57

ey

Update article.md

eb15c07

Update article.md

73fb0b3

Primer dia de trabajo. Completo, aunque necesita revision, hay cosas con las que no estoy seguro

javascript-translate-bot added the review needed label Sep 7, 2020

iliakan mentioned this pull request Sep 8, 2020

Spanish Translation Progress #17

Open

vplentinax requested changes Sep 8, 2020

View reviewed changes

javascript-translate-bot added changes requested and removed review needed labels Sep 8, 2020

vplentinax requested a review from joaquinelio September 8, 2020 02:43

vplentinax requested changes Sep 8, 2020

View reviewed changes

Comment thread 5-network/07-url/article.md Outdated

Comment thread 5-network/07-url/article.md Outdated

joaquinelio requested changes Sep 8, 2020

View reviewed changes

Agus-c and others added 14 commits September 17, 2020 17:20

Update 5-network/07-url/article.md

f4a447d

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

Update 5-network/07-url/article.md

745b03c

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 5-network/07-url/article.md

646464f

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

Update 5-network/07-url/article.md

b84cccc

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

Update 5-network/07-url/article.md

45fbab4

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 5-network/07-url/article.md

d6c7495

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

Update 5-network/07-url/article.md

78088ea

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

Update 5-network/07-url/article.md

3814ba8

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

Update 5-network/07-url/article.md

f615d7b

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

Update 5-network/07-url/article.md

aa98c6d

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

Update 5-network/07-url/article.md

a760ce6

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

Update 5-network/07-url/article.md

0b59416

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

Update 5-network/07-url/article.md

78f9b37

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

Update 5-network/07-url/article.md

643ecd4

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

Update 5-network/07-url/article.md

c467e08

Co-authored-by: joaquinelio <joaquinelio@gmail.com>

javascript-translate-bot added review needed and removed changes requested labels Sep 17, 2020

javascript-translate-bot requested a review from a team September 17, 2020 20:29

joaquinelio approved these changes Sep 20, 2020

View reviewed changes

javascript-translate-bot added needs +1 and removed review needed labels Sep 20, 2020

vplentinax approved these changes Sep 20, 2020

View reviewed changes

vplentinax added ready to merge and removed needs +1 labels Sep 20, 2020

joaquinelio merged commit 304689c into javascript-tutorial:master Sep 20, 2020

	- `url` -- La URL completa o único path (si base está establecido, mira debajo),
	- `url` -- La URL completa, o solamente la ruta si se establece base (mira debajo),

	Existe un estándar [RFC3986](https://tools.ietf.org/html/rfc3986) que define cuales caracteres son permitidos en URLs y cuáles no.
	Existe un estándar [RFC3986](https://tools.ietf.org/html/rfc3986) que define cuáles caracteres son permitidos en URLs y cuáles no.

Uh oh!

Conversation

Agus-c commented Sep 7, 2020

Uh oh!

CLAassistant commented Sep 7, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

vplentinax left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

javascript-translate-bot commented Sep 8, 2020

Uh oh!

vplentinax left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

javascript-translate-bot commented Sep 8, 2020

Uh oh!

joaquinelio left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

joaquinelio Sep 8, 2020

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

joaquinelio Sep 8, 2020

Choose a reason for hiding this comment

Uh oh!

Uh oh!

javascript-translate-bot commented Sep 8, 2020

Uh oh!

joaquinelio commented Sep 8, 2020

Uh oh!

Agus-c commented Sep 17, 2020

Uh oh!

joaquinelio left a comment

Choose a reason for hiding this comment

Uh oh!

vplentinax left a comment

Choose a reason for hiding this comment

Uh oh!

javascript-translate-bot commented Sep 20, 2020

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

CLAassistant commented Sep 7, 2020 •

edited

Loading