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

Add a link to help with Forking a repo #173

Merged
merged 2 commits into from
Mar 19, 2017

Conversation

jarvisluong
Copy link
Contributor

I also fixed some spelling mistakes and some Vietnamese writing styles.

I also fixed some spelling mistakes and some Vietnamese writing styles.
Copy link
Contributor

@dangkhoasdc dangkhoasdc left a comment

Choose a reason for hiding this comment

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

so far so good, I just find a typo in your changes. Please fix this and @letientai299 is gonna merge your PR. :)

@@ -216,7 +219,7 @@ Ok, bây giờ bạn có thể bắt đầu:
## Về nguồn video

Một vài video chỉ xem được khi bạn tham gia vào các lớp học online trên Coursera, EdX, hay Lynda.com. Các lớp đó được gọi là MOOC.
Đôi khi các lớp không có sẵn, và bạn phải đợi một vài tháng đến khi chúng được mở lại, do đó bạn không thể truy cập vào video được.
Đôi khi các lớp chư mở, và bạn phải đợi một vài tháng đến khi chúng được mở lại, do đó bạn không thể truy cập vào video được.
Copy link
Contributor

Choose a reason for hiding this comment

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

chưa mở :D

@letientai299
Copy link
Contributor

Thanks @dangkhoasdc for your help on reviewing.

@jarvisluong
Copy link
Contributor Author

@dangkhoasdc @letientai299 I have fixed the typo :)

@letientai299
Copy link
Contributor

@jwasham We have done the review. Please merge this PR.

@jwasham jwasham merged commit 3421790 into jwasham:master Mar 19, 2017
@jwasham
Copy link
Owner

jwasham commented Mar 19, 2017

Thanks!

@jarvisluong jarvisluong deleted the vietnamese/add-link-forking branch March 19, 2017 19:38
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