Skip to content

docs: Tooling > Single File Components の翻訳 #88

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

Merged
merged 4 commits into from
Sep 24, 2020
Merged

docs: Tooling > Single File Components の翻訳 #88

merged 4 commits into from
Sep 24, 2020

Conversation

takumiz19
Copy link
Collaborator

Description of Problem

#39 Tooling > Single File Components の翻訳
https://v3.vuejs.org/guide/single-file-component.html

Proposed Solution

https://jp.vuejs.org/v2/guide/single-file-components.html
こちらをベースに翻訳しました

Additional Information

特に無し

@takumiz19
Copy link
Collaborator Author

takumiz19 commented Sep 24, 2020

こちらのドキュメント内でwebpackという単語の表記揺れがあるのですがそのままで大丈夫でしょうか?

@netlify
Copy link

netlify bot commented Sep 24, 2020

Deploy preview for vuejs-v3-ja-doc-preview ready!

Built with commit 73a0c2f

https://deploy-preview-88--vuejs-v3-ja-doc-preview.netlify.app

@takumiz19 takumiz19 changed the title Sfcdoc to ja docs: Tooling > Single File Components の翻訳 Sep 24, 2020
Copy link
Member

@kazupon kazupon left a comment

Choose a reason for hiding this comment

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

レビュー 👀 コメント 💬 しました。
コメントの部分、対応お願いします! 🙏

@kazupon
Copy link
Member

kazupon commented Sep 24, 2020

こちらのドキュメント内でwebpackという単語の表記揺れがあるのですがそのままで大丈夫でしょうか?

本家の vuejs.org の方も表記ゆれがあるので、一旦このままでさせてください。 🙏

翻訳トーンの調整による修正

Co-authored-by: kazuya kawaguchi <kawakazu80@gmail.com>
@takumiz19
Copy link
Collaborator Author

@kazupon
レビューいただきありがとうございます!
ご指摘いただいた部分対応いたしましたのでもう一度ご確認お願いいたします!

@kazupon
Copy link
Member

kazupon commented Sep 24, 2020

対応ありがとうございました!
確認しました。
LGTMです!
CIでエラーが起きていますが、翻訳作業に支障がない(このPRとは別事象)ので、マージします!

翻訳ありがとうございました!

@kazupon kazupon merged commit c7e7f2e into vuejs-jp:lang-ja Sep 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants