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

pt-br docs #18

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

pt-br docs #18

wants to merge 11 commits into from

Conversation

paulojeronimo
Copy link

@paulojeronimo paulojeronimo commented Jul 24, 2017

Hello @anders94!

I would like to say that I loved your blockchain-demo. So, I decided to contribute to it, creating and sharing with my Brazilian friends, docs in our language (pt-br). I created these docs using asciidoctor. I created a GitHub Page where, at this moment, there is a document. Also, there is a PDF version of this document (built from the same sources). If you like it, I would love to participate in the project by offering him these documents.

This week I'm working on produce some videos (in Brazilian Portuguese). They will finish a blockchain presentation that I made to a Java User Group here, in Brasil. Then, I will publish they to YouTube and, ater that, I'll create closed captions for it, in English. I have some ideas about how can I improve the project. Some of them involve updating the code you wrote. I would like to continue to develop the project, along with you.

Thank you!

README.md Outdated
@@ -1,6 +1,9 @@
# Blockchain Demo
A web-based demonstration of blockchain concepts.

_NOTE: If you want to read a documentation in Brazilian Portuguese [click this link](https://blockchain4devs.github.io/blockchain-demo/docs/pt-br/).
Copy link
Owner

Choose a reason for hiding this comment

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

Can we do this in another way? Ideally there is a markdown version of the docs that we look to locally on GitHub. Thoughts?

@paulojeronimo
Copy link
Author

@anders94 Agree! I reverted README.md to original and added README.pt-br.md

README.md Outdated
@@ -1,5 +1,6 @@
# Blockchain Demo
A web-based demonstration of blockchain concepts.
_(If you want to read this in Brazilian Portuguese [click this link](docs/pt-br/index.html))_.
Copy link
Owner

Choose a reason for hiding this comment

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

Let's change this to simply:

_[Brazilian Portuguese version](docs/pt-br/index.html))_

Copy link
Author

Choose a reason for hiding this comment

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

Could be:
Brazilian Portuguese version: README.pt-br.md
For me, no problem if this reference isn't inside README.md. In this case, no more modifications are needed. It's up to you! 😄
Do we agree?

@paulojeronimo
Copy link
Author

Hi, @anders94! Do you need some my iteration in this PR in order to approve it? Is it ok for you?

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.

2 participants