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.

Agus-c added3 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

Copy link

CLAassistant commentedSep 7, 2020•
edited
Loading

All committers have signed the CLA.

iliakan mentioned this pull request

Sep 8, 2020

Spanish Translation Progress#17

Open

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.

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!

5-network/07-url/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:43

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.

Disculpa, olvidé estas correcciones...

5-network/07-url/article.md OutdatedShow resolvedHide resolved

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 👻

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.

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

5-network/07-url/article.md OutdatedShow resolvedHide resolved

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


		```js
		new URL(url, [base])
		```

		- `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),

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.

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 (sibase está establecido,mira debajo),
	-`url` -- La URL completa, osolamente la rutasise establece base (mira debajo),

5-network/07-url/article.md OutdatedShow resolvedHide resolved

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


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

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.

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 definecuales caracteres son permitidos en URLs y cuáles no.
	Existe un estándar[RFC3986](https://tools.ietf.org/html/rfc3986) que definecuáles caracteres son permitidos en URLs y cuáles no.

5-network/07-url/article.md OutdatedShow resolvedHide resolved

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 👻

Copy link

Member

joaquinelio commentedSep 8, 2020

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

Agus-cand others added14 commits

September 17, 2020 17:20