search

javascript-tutorial / pt.javascript.info

Tutorial de JavaScript Moderno em Português
https://javascript.info
Other
227 stars 112 forks source link

Extending built-in classes #413

Closed danilolmc closed 9 months ago

javascript-translate-bot commented 10 months ago

Error: the article already has PR number in the Progress Issue #1, it's 411 :interrobang:

danilolmc commented 10 months ago

/done

jonnathan-ls commented 9 months ago

I just found two points to be reviewed, so I will pass to you @jonnathan-ls the responsibility of reviewing the rest.

Adicionei minha aprovação!

Por mais que ainda tenha solicitado alterações @nazarepiedady, acabou marcando a opção de review como comentário ... mas considerando que o @danilolmc já adicionou as alterações, entendo que a solicitação foi atendida, configurando como aprovada, então seguirei com o merge deste PR.

Quando adicionar uma revisão que tenha solicitações de mudança @nazarepiedady, procure marcar a opção:

Request changes Submit feedback that must be addressed before merging.

Assim, o github registra as considerações e o BOT configurado do repositório consegue atuar, acho que foi por isso que ele acabou não atualizando o status deste PR, já que registrou como comentário em vez de request changes.

image

Obs.: Marcar como request changes e approved, mantém o vinculo de detalhes da aprovação associadas ao PR 01

javascript-translate-bot commented 9 months ago

Error: the article has another PR number in the Progress Issue #1, it's 411, not 413 :interrobang:

jonnathan-ls commented 9 months ago

Error: the article has another PR number in the Progress Issue 1, it's 411, not 413 ⁉️

Updated!