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 Japanese translation for Concurrency and async / await #2058

Merged
merged 10 commits into from
Feb 7, 2021

Conversation

tokusumi
Copy link
Contributor

@tokusumi tokusumi commented Sep 15, 2020

This PR translates the async.md to Japanese.
This section is technical, descriptive and long. I translated loosely to make it easier to read for Japanese.

Feel free to comment your advise or approve.

@codecov
Copy link

codecov bot commented Sep 15, 2020

Codecov Report

Merging #2058 (d9d4c23) into master (f031973) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #2058   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          243       243           
  Lines         7419      7419           
=========================================
  Hits          7419      7419           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cb40e27...d9d4c23. Read the comment docs.

@tokusumi tokusumi mentioned this pull request Sep 15, 2020
@github-actions
Copy link
Contributor

📝 Docs preview for commit a8b57e8 at: https://5f61207d65823e35bb3a91dd--fastapi.netlify.app

Copy link
Contributor

@komtaki komtaki left a comment

Choose a reason for hiding this comment

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

Thank you for translating the difficult page. 👍

I commented on some of the things that could be improved.

docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
Co-authored-by: Taki Komiyama <39375566+komtaki@users.noreply.github.com>
@github-actions
Copy link
Contributor

📝 Docs preview for commit 25b888b at: https://5f6f14283bbe865e72b6fe5d--fastapi.netlify.app

Copy link
Contributor

@komtaki komtaki left a comment

Choose a reason for hiding this comment

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

Thank you. LGTM 👍

@tiangolo tiangolo added awaiting-review lang-all Translations lang-ja Japanese translations labels Oct 15, 2020
@github-actions
Copy link
Contributor

📝 Docs preview for commit af68dca at: https://5f8c3fb834376717567260a3--fastapi.netlify.app

Copy link
Contributor

@sh0nk sh0nk left a comment

Choose a reason for hiding this comment

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

It is a very interesting and impressive document, and I'm super appreciated with your hard work of the translation with this big volume!

Especially the original burger stories are written in very story telling style with a lot of Emojis, so I believe even the translation could be in more spoken form in Japanese, which would attract more people who are beginners of the FastAPI world!

I haven't changed so much, but tried to make it a bit easy to read. So please consider for the suggestions!

docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
Copy link
Contributor

@sh0nk sh0nk left a comment

Choose a reason for hiding this comment

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

Thanks for the deep look. Replied to the comment and just found a single dead link.

docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
docs/ja/docs/async.md Outdated Show resolved Hide resolved
Co-authored-by: Sho Nakamura <sh0nk.developer@gmail.com>
@github-actions
Copy link
Contributor

📝 Docs preview for commit 4d8bbe7 at: https://6003d346685d2641bcd72540--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit 63a18fd at: https://6003d4838a2bbac9e5563254--fastapi.netlify.app

Co-authored-by: Sho Nakamura <sh0nk.developer@gmail.com>
@github-actions
Copy link
Contributor

📝 Docs preview for commit 3226a75 at: https://6004282aab6d721364aa7616--fastapi.netlify.app

Co-authored-by: Sho Nakamura <sh0nk.developer@gmail.com>
@github-actions
Copy link
Contributor

📝 Docs preview for commit e862e2a at: https://60042d6d59b42bfb3e1bc214--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit d9d4c23 at: https://6004308fe6f31a277e286917--fastapi.netlify.app

@tokusumi
Copy link
Contributor Author

@sh0nk Thank you for your review!
I applied most your suggestions modified a little.
Could you review again? (Sorry for inconvenience that my commits are separated in disorders)

Copy link
Contributor

@sh0nk sh0nk left a comment

Choose a reason for hiding this comment

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

Looks very good to me 👍 Thank you for taking the suggestions with patience and persistence, even though the number is so huge!

Again, this doc must be a very impressive one for beginners of FastAPI to learn how and what is async! Not only that, I believe this is also good for people who are not the FastAPI users.

@tiangolo tiangolo changed the title Add japanese translation for async.md 🌐 Add Japanese translation for Concurrency and async / await Feb 7, 2021
@tiangolo
Copy link
Member

tiangolo commented Feb 7, 2021

Awesome! Thanks a lot for all the work on this! 🤓 💪 🙇

And thank you @sh0nk and @komtaki for the thorough reviews! 🍰 ☕

@tiangolo tiangolo merged commit faec713 into fastapi:master Feb 7, 2021
@tokusumi tokusumi deleted the translation/add-japanese-async branch May 5, 2021 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved-2 lang-all Translations lang-ja Japanese translations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants