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.

Sign up for GitHub

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

Jump to bottom

Tradução da navbar de algumas paginas que já estão na produção #131

Merged
merged 3 commits into from Mar 1, 2019
Merged

Tradução da navbar de algumas paginas que já estão na produção #131

merged 3 commits into from Mar 1, 2019

Conversation

WeslleyNasRocha
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented Feb 26, 2019 •
edited

Deploy preview for pt-br-reactjs ready!

Built with commit b854774

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

@halian-vilela
Copy link
Collaborator

@WeslleyNasRocha

Ótima iniciativa! Creio que você pode seguir a Checklist da Issue principal --> #1

Valeu!

@WeslleyNasRocha
Copy link
Contributor Author

@WeslleyNasRocha

Ótima iniciativa! Creio que você pode seguir a Checklist da Issue principal --> #1

Valeu!

Boa ideia, assim que tiver um tempinho crio a issue pra controlar certinho

@eduardomoroni eduardomoroni added the needs review A pull request ready to be reviewed label Feb 27, 2019
tibuurcio
tibuurcio requested changes Feb 27, 2019
View reviewed changes
Copy link
Collaborator

@tibuurcio tibuurcio left a comment

Choose a reason for hiding this comment

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

Opa, apenas pequenas correções em relação ao glossário :) Valeu!

content/docs/nav.yml Outdated Show resolved Hide resolved
content/docs/nav.yml Outdated Show resolved Hide resolved
content/docs/nav.yml Outdated Show resolved Hide resolved
@tibuurcio tibuurcio 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 27, 2019
tibuurcio and others added 2 commits February 27, 2019 19:49
@tibuurcio @WeslleyNasRocha
Update content/docs/nav.yml
2a47056 
Co-Authored-By: WeslleyNasRocha <weslleytato.12@gmail.com>
@tibuurcio @WeslleyNasRocha
Apply suggestions from code review
b854774 
Co-Authored-By: WeslleyNasRocha <weslleytato.12@gmail.com>
@halian-vilela halian-vilela mentioned this pull request Feb 28, 2019
Closed
@eduardomoroni eduardomoroni added feedback addressed and removed needs +1 Needs another approval before merging needs author response Changes requested needing author's input labels Feb 28, 2019
@eduardomoroni
Copy link
Member

@tibuurcio as alterações foram aceitas. Pode dar mais uma olhada brow?

@jhonmike jhonmike mentioned this pull request Feb 28, 2019
Merged
4 tasks
@tibuurcio
Copy link
Collaborator

Estamos com um PR aberto para adicionar o @jhonmike como mantenedor do projeto e como ele já deu o approve também, estou dando merge nesse PR 😄

@tibuurcio tibuurcio merged commit 62fd1ea into reactjs:master Mar 1, 2019
@tibuurcio tibuurcio added 🌐 translated Translation is completed for this page and removed feedback addressed labels Mar 1, 2019
glaucia86
glaucia86 reviewed Mar 3, 2019
View reviewed changes
Copy link
Member

@glaucia86 glaucia86 left a comment

Choose a reason for hiding this comment

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

@WeslleyNasRocha tenta fazer as alterações necessárias. Assim que os fizer, nos sinalize para que possamos aprovar o quanto antes a sua contribuição.
Muito obrigada! :)

cc: @eduardomoroni @halian-vilela @tibuurcio @cezaraugusto se puderem dar uma olhada também, para que possamos dar aprovação necessária. :)

content/docs/nav.yml Show resolved Hide resolved
content/docs/nav.yml Show resolved Hide resolved
content/docs/nav.yml Show resolved Hide resolved
content/docs/nav.yml Show resolved Hide resolved
content/docs/nav.yml Show resolved Hide resolved
content/docs/nav.yml Show resolved Hide resolved
content/docs/nav.yml Show resolved Hide resolved
content/docs/nav.yml Show resolved Hide resolved
content/docs/nav.yml Show resolved Hide resolved
content/docs/nav.yml Show resolved Hide resolved
@glaucia86 glaucia86 added needs review A pull request ready to be reviewed needs author response Changes requested needing author's input and removed 🌐 translated Translation is completed for this page labels Mar 3, 2019
@halian-vilela
Copy link
Collaborator

halian-vilela commented Mar 4, 2019 •
edited

@WeslleyNasRocha tenta fazer as alterações necessárias. Assim que os fizer, nos sinalize para que possamos aprovar o quanto antes a sua contribuição.
Muito obrigada! :)

cc: @eduardomoroni @halian-vilela @tibuurcio @cezaraugusto se puderem dar uma olhada também, para que possamos dar aprovação necessária. :)

Oi @glaucia86, vi que você removeu os espaços que tem antes de cada title. Analisando o arquivo original, eu acho que essa identação é importante para a organização hierárquica do YML, pode dar uma conferida?

Talvez possa ter sido seu editor tb, que removeu automaticamente.

Valeu!

@jhonmike
Copy link
Member

jhonmike commented Mar 6, 2019

Olá pessoal acho q ouvi um conflito aqui hahaha, @glaucia86 se vc notar no PR #135 eu já fiz o resto das traduções que não tinha nesse arquivo.

O objetivo desse PR do @WeslleyNasRocha era traduzir apenas os títulos que já tinham paginas traduzidas, e não todos eles, por isso esse PR foi "mergeado" e dado como DONE

@jhonmike jhonmike removed needs author response Changes requested needing author's input needs review A pull request ready to be reviewed labels Mar 6, 2019
@glaucia86 glaucia86 mentioned this pull request Mar 6, 2019
Closed
10 tasks
@WeslleyNasRocha
Copy link
Contributor Author

Isso mesmo @jhonmike , até porque é meio que o trabalho de quem fez a tradução da página, traduzir o sidebar

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@glaucia86 glaucia86 glaucia86 left review comments

@jhonmike jhonmike jhonmike approved these changes

@tibuurcio tibuurcio tibuurcio approved these changes

Assignees

@tibuurcio tibuurcio

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@WeslleyNasRocha @halian-vilela @eduardomoroni @tibuurcio @jhonmike @glaucia86