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

Issue#68 translate tutorials #120

Closed
wants to merge 0 commits into from
Closed

Issue#68 translate tutorials #120

wants to merge 0 commits into from

Conversation

raulfdm
Copy link
Contributor

@raulfdm raulfdm commented Feb 21, 2019

Continuando a issue #68, aqui está a tradução faltante.

Ainda amanhã pretendo fazer eu mesmo uma revisão de quando comecei e de alguns pontos que marquei para repensar. Porém, acredito que já possa dar início à revisão.

@eduardomoroni @halian-vilela

@netlify
Copy link

netlify bot commented Feb 21, 2019

Deploy preview for pt-br-reactjs ready!

Built with commit 4efc4b7

https://deploy-preview-120--pt-br-reactjs.netlify.com

Copy link
Contributor

@Luanf Luanf left a comment

Choose a reason for hiding this comment

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

Amanha retomo o review, segue alguns detalhes que encontrei ate o momento

content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
Copy link
Contributor

@Luanf Luanf left a comment

Choose a reason for hiding this comment

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

Continuando o review...

content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
@halian-vilela
Copy link
Collaborator

Oi @Luanf , blz?

Obrigado pela revisão. Vi que você está adicionando as correções no comentário, talvez não tenha visto que é possível já corrigir no texto real e essa correção irá se transformar em uma sugestão de commit que autor pode só aceitar direto. É um jeito bem mais fácil de fazer a revisão.

Basta usar esse botão indicado aqui:
issue_68_translate_tutorials_by_raulfdm_ pull_request__120 _reactjs_pt-br_reactjs_org

Dá uma olhada em como eu fiz aqui --> #117 que você vai sacar.

Valeu!

@halian-vilela halian-vilela added the needs review A pull request ready to be reviewed label Feb 22, 2019
@Luanf
Copy link
Contributor

Luanf commented Feb 22, 2019

Opa @halian-vilela, ótimo!
Tinha visto essa feature mas estava um pouco hesitante de usar. 😊
Fiz alguns edits com sugestões e vou continuar revisando na medida que conseguir.
Obrigado!

Copy link
Contributor

@Luanf Luanf left a comment

Choose a reason for hiding this comment

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

Terminei uma primeira passada pelo arquivo todo, corrigi typos, equívocos na construção de frases e algumas incoerências.

content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
@halian-vilela halian-vilela added needs author response Changes requested needing author's input needs +1 Needs another approval before merging and removed needs review A pull request ready to be reviewed labels Feb 23, 2019
@raulfdm
Copy link
Contributor Author

raulfdm commented Feb 23, 2019

@Luanf corrigi suas correções/sugestões e fiz algumas pequenas alterações em alguns termos (ea32a0c)

@halian-vilela halian-vilela removed the needs author response Changes requested needing author's input label Feb 23, 2019
content/tutorial/tutorial.md Outdated Show resolved Hide resolved
@glaucia86
Copy link
Member

@raulfdm conseguiu fazer as alterações necessárias? Se precisar de ajuda, por favor, nos sinalize! ;)

@glaucia86 glaucia86 added needs author response Changes requested needing author's input needs review A pull request ready to be reviewed and removed needs +1 Needs another approval before merging labels Mar 3, 2019
@raulfdm
Copy link
Contributor Author

raulfdm commented Mar 3, 2019

@raulfdm conseguiu fazer as alterações necessárias? Se precisar de ajuda, por favor, nos sinalize! ;)

Já está tudo certo 🤔
To esperando os próximos passos que nem sei quais são.

@halian-vilela
Copy link
Collaborator

halian-vilela commented Mar 4, 2019

@raulfdm
Eu já estava quase fazendo o merge mas vi que o arquivo está com vários conflitos. Pode conferir se sua tradução não conflitou com parte do arquivo que já estava traduzida? Creio que você sobrescreveu o que já estava revisado e mergeado das duas issues anteriores.

cc: @glaucia86 @Luanf

@glaucia86
Copy link
Member

@raulfdm só tenta resolver os conflitos que estão aparecendo no arquivo. Pois tou tentando aprovar e não consigo. Tá dando problema de merge ;)

cc: @halian-vilela

@glaucia86 glaucia86 removed the needs review A pull request ready to be reviewed label Mar 5, 2019
@raulfdm
Copy link
Contributor Author

raulfdm commented Mar 5, 2019

To aqui resolvendo os conflitos e notei que... O @halian-vilela fez ou refez a tradução que referente ao que me passaram pra fazer e ela já está na master.

A minha dúvida é: esse PR é mesmo necessário?

@glaucia86

@halian-vilela
Copy link
Collaborator

To aqui resolvendo os conflitos e notei que... O @halian-vilela fez ou refez a tradução que referente ao que me passaram pra fazer e ela já está na master.

A minha dúvida é: esse PR é mesmo necessário?

@glaucia86

Oi @raulfdm, foi isso que eu vi em relação aos seus conflitos. Vc chegou a fazer o pull do Master antes de iniciar? Talvez vc tenha iniciado antes e não tenha visto o momento do merge da issue #68, mas é que na verdade, a parte do arquivo que estava faltando traduzir era somente as linhas 347 até 607, pois era a diferença que tinha sobrado entre a primeira (traduzida pelo @eduardomoroni ) e a segunda metade do arquivo (traduzida por mim). Dá uma lida lá no meu comentário (#68 (comment)) que vc vai entender.

Mas creio que como vc traduziu a parte que faltava, basta deixar o resto como estava e a gente já faz o merge!

Ficou mais claro?

Valeo!

@raulfdm
Copy link
Contributor Author

raulfdm commented Mar 5, 2019

@halian-vilela Realmente, entendi errado e acabei fazendo 367 -> final. Mybad.

Abri outro MR pra ficar mais fácil (#142).

Copiei só as linhas que faltavam e colei no que está na master do repositório oficial conforme comentado.

Malz o mal entendido! 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs author response Changes requested needing author's input
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants