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

Correct pt-BR translation #52

Merged
merged 1 commit into from May 5, 2015

Conversation

megazord
Copy link
Contributor

I've also correct small typos at the google docs spreadsheet.

@megazord
Copy link
Contributor Author

@igorcosta since you did the original translation, fell free to comment on this PR. :-)

@cadocruz
Copy link

@megazord, good work.

@megazord
Copy link
Contributor Author

megazord commented May 2, 2015

Could someone take a look here in order to integrate this PR? The current pt-BR translation has a lot of errors.

@igorcosta
Copy link
Contributor

Everything depends on single point of view. @megazord you did a good job.

@jboner
Copy link
Member

jboner commented May 3, 2015

Is it ready to go?

@megazord
Copy link
Contributor Author

megazord commented May 5, 2015

@jboner Yep!

@jboner
Copy link
Member

jboner commented May 5, 2015

Great. Thanks a lot. @rasummer

jboner added a commit that referenced this pull request May 5, 2015
@jboner jboner merged commit 4815f32 into reactivemanifesto:master May 5, 2015
@marcospereira
Copy link

@jboner any idea about when it will be updated on website?

@jboner
Copy link
Member

jboner commented May 8, 2015

When @rasummer or @jroper will get a chance to push it.

@jroper
Copy link
Member

jroper commented May 8, 2015

@marcospereira these fixes need to be copied to

https://github.com/reactivemanifesto/website-manifesto/blob/master/app/views/ptBR/manifesto.scala.html

They also need to be made on the PDF. @rasummer usually does this stuff, I'm not sure what his process is.

@rasummer
Copy link
Member

rasummer commented May 8, 2015

I may have time later today to make both the HTML and PDF changes, if not then it will be early next week.

@marcospereira
Copy link

@rasummer and @jroper thanks!

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.

None yet

7 participants