Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inseridas URLs em todos (veja ... ) #314

Open
wants to merge 16 commits into
base: portuguese-translation
Choose a base branch
from

Conversation

wfrsilva
Copy link
Contributor

Antes de enviar seu Pull Request

Faça o check abaixo para ver se o PR está no melhor padrão possível.
Note que o conteúdo enviado será revisado antes de ser aceito, em geral blocos maiores de tradução levarão mais tempo
para serem aceitos (veja #68). A qualidade do material é um compromisso nosso.

  • Verifique se os jargões estão coerentes com o WORDREFERENCE
  • Revise a tradução com carinho 💕 -- seu nome constará na parte que você traduziu.
  • Certifique-se que a branch de sua tradução contém as últimas atualizações da branch portuguese-translation do repositório principal.
  • Ao terminar sua parte da tradução, faça um Pull Request para o branch portuguese-translation.
  • Caso o Pull Request for o último (finalização de capítulo), use o número da issue como referência e faça uso dessas keywords.

Inseridas URLs nos tópicos do capitulo 1
(Foi até onde li, pretendo ir lendo e atualizando as URLs)
[Capítulo 2: Por dentro do JavaScript]
[Capítulo 3: Dentro do YDKJS]
nos títulos somente, seus itens ainda não!
Parenteses de URL funcoes corrigida
9 -> (
informaçoes corrigido informações
veja os primeiros três capítulos dos títulos dessa série 
[*Escopos & Clausuras*]
(https://github.com/wfrsilva/You-Dont-Know-JS/blob/portuguese-translation/up%20%26%20going/ch3.md#escopos--clausuras).

Capítulo 3: Dentro do YDKJS
Escopos & Clausuras
Colocadas URLs em todos os topicos 
do capitulo 2:
"Valores e tipos" até "Não-JavaScript"
URL Variaveis do capitulo 2 com erro de formatação
Faltou ")" no final da URL
Copy link
Owner

@cezaraugusto cezaraugusto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oi @wfrsilva, os links estão levando para o seu fork ao invés deste repo. poderia atualizar?

@wfrsilva
Copy link
Contributor Author

wfrsilva commented Dec 15, 2019 via email

Trocadas as URLs que estavam equivocadasmente com
wfrsilva para cezaraugusto
Inserindo URLs para o dominio wfrsilva no capitulo 2
URLs wfrsilva
URLs Faltantes
Copy link
Collaborator

@gabsprates gabsprates left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Falta corrigir em:

  • up & going/ch2.md
  • up & going/toc.md

Dica: os links podem estar em caminhos relativos.

Também precisa de rebase pra corrigir conflitos.


PS.: Particularmente, acho essas modificações um pouco complicadas de se manter: a atualização de um títulos e seções podem fazer essas referências se perderem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants