Skip to content

update docs/ja #978

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 28, 2018
Merged

update docs/ja #978

merged 4 commits into from
Sep 28, 2018

Conversation

kazupon
Copy link
Member

@kazupon kazupon commented Sep 23, 2018

No description provided.

@kazupon
Copy link
Member Author

kazupon commented Sep 23, 2018

/ping @38elements 🙏

@38elements
Copy link
Contributor

返信が遅くなって大変申し訳ございません。
Lintでエラーになっています。
私はVue Test Utilsの日本語ドキュメントを更新しないので
互換性を重視するグループの人がVue Test Utilsの日本語ドキュメントを
レビューも含めて更新していただきたいと思います。
よろしくお願いします。

@kazupon
Copy link
Member Author

kazupon commented Sep 25, 2018

確認ありがとうございます。
38elementsさんが今後、日本語ドキュメントを対応しないという点、了解しました。
Vue Test Utilsの日本語ドキュメントに興味がある方にお願いしたいと思います。

@potato4d
Copy link
Contributor

@kazupon 誰でもレビューサクッと身始められるようにDescriptionに対応する英語ファイルをへのリンクを貼っておいてもらえると助かります!

Copy link

@tell-k tell-k left a comment

Choose a reason for hiding this comment

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

@@ -131,6 +131,20 @@ mount(Component, {
})
```

### スタブコンポーネント

グローバルまたは `stubs` オプションを使用してローカルに登録されたコンポーネントを上書きできます:
Copy link

Choose a reason for hiding this comment

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

  • by using the stubs option は その前の一文全体に掛かってると思われます。なので下記のようにするのはどうでしょうか。
修正案:  `stubs` オプションを使用して、グローバルまたはローカルに登録されたコンポーネントを上書きできます

Copy link
Member Author

Choose a reason for hiding this comment

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

あ、確かにそうですね。そのように修正します!


mount(Component, {
// 空スタブによってグローバルに登録されたコンポーネントを
// 解決します
Copy link

Choose a reason for hiding this comment

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

  • globally-registered-component は スタブの名前なので、そのまま利用した方が良さそうに思いました。
修正案: globally-registered-component を空のスタブとして解決します。

Copy link
Member Author

Choose a reason for hiding this comment

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

確かにそうですね。そのように修正します!

@kazupon
Copy link
Member Author

kazupon commented Sep 26, 2018

@tell-k
レビューありがとうございました!
これから修正しますね。

@kazupon
Copy link
Member Author

kazupon commented Sep 26, 2018

@potato4d
そうですね。確かに無のdescriptionだと分からないですね。

自分は、以下のようにコミットから原文と翻訳を対比しながら見れるからいいかなと思っていたんですが、説明しておかないと分からないですね。。。
b570a1a

以後、気をつけます。

```

::: 注意
`propsData` は Vue Test Utils のマウティングオプションではなく [Vue API](https://vuejs.org/v2/api/#propsData) です。
Copy link

Choose a reason for hiding this comment

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

誤字 👀
マウティング → マウンティング

Copy link
Member Author

Choose a reason for hiding this comment

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

ありがとうございます!

@kazupon
Copy link
Member Author

kazupon commented Sep 26, 2018

done
特に修正内容に問題がなければ、セルフーマージします。

@kazupon
Copy link
Member Author

kazupon commented Sep 26, 2018

あ、自分権限ないからEdd氏にお願いするという感じでしたね。。。

@eddyerburgh
Copy link
Member

Do you want me to merge @kazupon ?

@kazupon
Copy link
Member Author

kazupon commented Sep 28, 2018

@eddyerburgh
Yes, it's ok! 🙏

Copy link
Member

@eddyerburgh eddyerburgh 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 translating :)

@eddyerburgh eddyerburgh merged commit 2f4c073 into vuejs:dev Sep 28, 2018
@kazupon kazupon deleted the docs/ja branch September 28, 2018 05:57
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.

6 participants