Rest parameters and spread operator #73

luizbaldi · 2019-10-08T02:06:43Z

Adding translation for "Rest parameters and spread operator".

Feel free to share thoughts on the translation and let me know if any changes are needed :)

odsantos · 2019-10-08T04:50:16Z

Hi, hope to do it as soon as possible. Thanks. Cheers.

odsantos

Hi Luiz,

Once again a great translation.
Would you please have a look at the proposed changes?

Thanks.

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md

javascript-translate-bot · 2019-10-11T10:36:21Z

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

luizbaldi · 2019-10-11T12:30:00Z

Hi Luiz,

Once again a great translation.
Would you please have a look at the proposed changes?

Thanks.

Of course! Thanks for your review, gonna fix those changes asap :)

odsantos · 2019-10-11T13:25:32Z

Thanks. On Fri, Oct 11, 2019 at 1:29 PM, Luiz Baldi<notifications@github.com> wrote: look Of course! Thanks for you review, gonna fix those changes asap :) — You are receiving this because your review was requested. Reply to this email directly, view it on GitHub, or unsubscribe.

luizbaldi · 2019-10-20T19:35:08Z

/done

odsantos · 2019-10-20T23:59:28Z

Hi, I hope to do it as soon as possible. Thanks.

odsantos

Hi Luiz,

Would you please consider the following changes?

Thanks.

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md

javascript-translate-bot · 2019-10-28T06:13:52Z

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

luizbaldi · 2019-10-30T00:40:29Z

/done

odsantos

Hi Luiz,

It seems that your file has many error conflicts.
On such occasions, would you please check the English article for ways to solve them, and see if the translation might match that text as much as possible?

Besides the missing code, the translation could perhaps be taken as finished. Adding 'o' and 'do' where they may not exist, is not a concern.

Thank you.
Regards.

odsantos · 2019-11-01T00:37:19Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md

 ````

-As we remember, arrow functions don't have their own `this`. Now we know they don't have the special `arguments` object either.
+Como nos lembramos, funções _arrow_ não possuem seu próprio `this`. Agora, sabemos que elas também não possuem o especial `arguments`.


Please,

'... não possuem o seu próprio this.'

odsantos · 2019-11-01T00:56:26Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md

-alert(merged); // 0,3,5,1,2,8,9,15 (0, then arr, then 2, then arr2)
+alert(merged); // 0,3,5,1,2,8,9,15 (0, depois arr, depois 2, depois arr2)
 ```



Please,

The code above seems to contain an error conflict.
(Would you check the English website article (link above)?)

I don't what you mean by "error conflict". The line 192 looks exactly like the original one on the english version, as we can see here. Can you explain to me what exactly should I do here?

Please, check Spread operator - 6th code block:
The second array (arr2) is missing
And also formatting characters

odsantos · 2019-11-01T00:56:53Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md


 alert( Math.max(1, ...arr1, 2, ...arr2, 25) ); // 25
 ```



Please,

The code above seems to contain an error conflict.
(Would you check the English website article (link above)?)

I don't what you mean by "error conflict". The line 179 looks exactly like the original one on the english version, as we can see here. Can you explain to me what exactly should I do here?

Please, check Spread operator - 5th code block:
The second array (arr2) is missing

odsantos · 2019-11-01T01:00:32Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md

+Juntos, eles nos ajudam a efetuar a transição entre uma lista e um _array_ de parâmetros com facilidade.

-All arguments of a function call are also available in "old-style" `arguments`: array-like iterable object.
+Todos os argumentos de uma chamada de função também estão disponíveis na "moda antiga" `arguments`: objeto iterável tipo lista-genérica (_array-like_).


Please,

'... objeto iterável do tipo lista-genérica (array-like).'

odsantos · 2019-11-01T01:03:56Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md

 f(1); // 1
 ```
 ````



Please,

The code above seems to contain an error conflict.
So, in such situations would you check the English article (there is a link here) for ways to resolve the conflict?
From the text there, you might know how the Portuguese page would look.

I didn't get this "error conflicts" at all, sorry. Can you explain me better what you mean by that?

Sure Luiz, would you please simply copy and paste the code from the English site here The “arguments” variable? (Just above "Spread operator")

Please, remove all code inside, so that the strange characters might disappear, such as:

```js run (please leave there) (>>> insert the code here) ``` (please leave there)

(for all cases where conflicts exist, check the English site and copy/paste all needed, including over the strange characters to remove them)
Thanks.

odsantos · 2019-11-01T01:04:53Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md

 alert( Math.max(arr) ); // NaN
 */!*
 ```



Please,

The code above seems to contain an error conflict.
(In doubt, would you check the English website article (link here)?)

Same goes here.

Please, check Spread operator - 2nd code block:
The alert is missing.
Simply copy/paste, as in the example above.

odsantos · 2019-11-01T01:05:25Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md


 alert( Math.max(...arr1, ...arr2) ); // 8
 ```



Please,

The code above seems to contain an error conflict.
(Would you check the English website article (link above)?)

The second array (arr2) is missing:
Please, check Spread operator - 4th code block.

javascript-translate-bot · 2019-11-01T01:13:27Z

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

luizbaldi · 2019-11-16T12:51:03Z

/done

odsantos

Hi, would you also check the comments below?
Appreciated for your time.

odsantos · 2019-12-13T18:20:33Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md


 alert( sum(1, 2, 3, 4, 5) );
 ```



Hi,

The function brace needs to be closed.
(you can check it at Rest parameters and spread operator - 1st code block)
Thanks.

odsantos · 2019-12-13T18:23:45Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md

 alert( sumAll(1, 2) ); // 3
 alert( sumAll(1, 2, 3) ); // 6
 ```



Hi,

There is code missing here.
(you can check it at Rest parameters and spread operator - 2nd code block)
Thanks.

odsantos · 2019-12-13T18:25:19Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md


 showName("Julius", "Caesar", "Consul", "Imperator");
 ```



Hi,

There is code missing here, too.
(you can check it at Rest parameters and spread operator - 3rd code block)
Thanks.

odsantos · 2019-12-13T18:35:32Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md

 f(1); // 1
 ```
 ````



Sure Luiz, would you please simply copy and paste the code from the English site here The “arguments” variable? (Just above "Spread operator")

Please, remove all code inside, so that the strange characters might disappear, such as:

```js run (please leave there) (>>> insert the code here) ``` (please leave there)

(for all cases where conflicts exist, check the English site and copy/paste all needed, including over the strange characters to remove them)
Thanks.

odsantos · 2019-12-13T18:47:39Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md

 alert( Math.max(arr) ); // NaN
 */!*
 ```



Please, check Spread operator - 2nd code block:
The alert is missing.
Simply copy/paste, as in the example above.

odsantos · 2019-12-13T18:51:59Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md


 alert( Math.max(...arr1, ...arr2) ); // 8
 ```



The second array (arr2) is missing:
Please, check Spread operator - 4th code block.

odsantos · 2019-12-13T18:56:43Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md


 alert( Math.max(1, ...arr1, 2, ...arr2, 25) ); // 25
 ```



Please, check Spread operator - 5th code block:
The second array (arr2) is missing

odsantos · 2019-12-13T19:06:58Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md

-alert(merged); // 0,3,5,1,2,8,9,15 (0, then arr, then 2, then arr2)
+alert(merged); // 0,3,5,1,2,8,9,15 (0, depois arr, depois 2, depois arr2)
 ```



Please, check Spread operator - 6th code block:
The second array (arr2) is missing
And also formatting characters

odsantos · 2019-12-13T19:11:34Z

1-js/06-advanced-functions/02-rest-parameters-spread-operator/article.md


-So, for a string, `for..of` returns characters and `...str` becomes `"H","e","l","l","o"`. The list of characters is passed to array initializer `[...str]`.
+Então, para uma string, `for..of` retorna caracteres, e `...str` se torna `"O","l","á"`. A lista de caracteres é passada para o inicializador do _array_ `[...str]`



Please,

"... retorna caracteres, e ...str se torna em "O","l","á"."
(but do not format in bold)

odsantos

Hi Luiz,

Your translations continue with high quality, as always.

The links you are giving here are very appropriate. There you may click in "Raw" and copy the pieces of code to paste in your file.

Please, do not paste over your translations, but above or below. If such, you may then copy/paste your translations and avoid to translate a second time.

If you have time, I would also appreciate if in the end you could mark as /done, as the bot suggests, because possibly the bot might be able to notify me faster.

Thanks.

javascript-translate-bot · 2019-12-13T19:31:05Z

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

CLAassistant · 2020-05-15T19:00:41Z

All committers have signed the CLA.

Translate rest parameters and spread operator article

43a9816

javascript-translate-bot added the review needed label Oct 8, 2019

javascript-translate-bot requested a review from a team October 8, 2019 02:06

iliakan mentioned this pull request Oct 8, 2019

Portuguese Translation Progress #1

Open

odsantos requested changes Oct 11, 2019

View reviewed changes

javascript-translate-bot removed the review needed label Oct 11, 2019

javascript-translate-bot added the changes requested label Oct 11, 2019

Fix rest parameters based on review

85e4f29

javascript-translate-bot added review needed and removed changes requested labels Oct 20, 2019

javascript-translate-bot requested a review from a team October 20, 2019 19:35

luizbaldi requested review from odsantos and removed request for a team October 20, 2019 19:35

odsantos requested changes Oct 28, 2019

View reviewed changes

javascript-translate-bot removed the review needed label Oct 28, 2019

javascript-translate-bot added the changes requested label Oct 28, 2019

Add more fixes on rest parameters article

670a9f0

javascript-translate-bot added review needed and removed changes requested labels Oct 30, 2019

javascript-translate-bot requested a review from a team October 30, 2019 00:40

luizbaldi requested review from odsantos and removed request for a team October 30, 2019 00:40

odsantos requested changes Nov 1, 2019

View reviewed changes

javascript-translate-bot added changes requested and removed review needed labels Nov 1, 2019

Minor fixes on spread operators article

8cec7aa

luizbaldi requested a review from odsantos November 13, 2019 13:05

javascript-translate-bot added review needed and removed changes requested labels Nov 16, 2019

javascript-translate-bot requested a review from a team November 16, 2019 12:51

odsantos reviewed Dec 13, 2019

View reviewed changes

odsantos requested changes Dec 13, 2019

View reviewed changes

javascript-translate-bot removed the review needed label Dec 13, 2019

javascript-translate-bot added the changes requested label Dec 13, 2019

luizbaldi closed this May 19, 2020


		So, for a string, `for..of` returns characters and `...str` becomes `"H","e","l","l","o"`. The list of characters is passed to array initializer `[...str]`.
		Então, para uma string, `for..of` retorna caracteres, e `...str` se torna `"O","l","á"`. A lista de caracteres é passada para o inicializador do _array_ `[...str]`

Uh oh!

Rest parameters and spread operator #73

Rest parameters and spread operator #73

Uh oh!

Conversation

luizbaldi commented Oct 8, 2019

Uh oh!

odsantos commented Oct 8, 2019

Uh oh!

odsantos 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!

Uh oh!

javascript-translate-bot commented Oct 11, 2019

Uh oh!

luizbaldi commented Oct 11, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

odsantos commented Oct 11, 2019 via email

Uh oh!

luizbaldi commented Oct 20, 2019

Uh oh!

odsantos commented Oct 20, 2019

Uh oh!

odsantos 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!

Uh oh!

javascript-translate-bot commented Oct 28, 2019

Uh oh!

luizbaldi commented Oct 30, 2019

Uh oh!

odsantos left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

luizbaldi Nov 8, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

luizbaldi Nov 8, 2019 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

luizbaldi commented Oct 11, 2019 •

edited

Loading

luizbaldi Nov 8, 2019 •

edited

Loading

luizbaldi Nov 8, 2019 •

edited

Loading

odsantos left a comment •

edited

Loading

odsantos left a comment •

edited

Loading