-
Notifications
You must be signed in to change notification settings - Fork 668
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
update docs/ja #978
Conversation
/ping @38elements 🙏 |
返信が遅くなって大変申し訳ございません。 |
確認ありがとうございます。 |
@kazupon 誰でもレビューサクッと身始められるようにDescriptionに対応する英語ファイルをへのリンクを貼っておいてもらえると助かります! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kazupon さん。 レビューしてみました。LGTM with nits.
下記元ファイルの内容と、翻訳文を比較して確認いたしました。
- https://github.com/vuejs/vue-test-utils/blob/dev/docs/README.md
- https://github.com/vuejs/vue-test-utils/blob/dev/docs/api/options.md
- https://github.com/vuejs/vue-test-utils/blob/dev/docs/api/shallowMount.md
- https://github.com/vuejs/vue-test-utils/blob/dev/docs/guides/common-tips.md
docs/ja/guides/common-tips.md
Outdated
@@ -131,6 +131,20 @@ mount(Component, { | |||
}) | |||
``` | |||
|
|||
### スタブコンポーネント | |||
|
|||
グローバルまたは `stubs` オプションを使用してローカルに登録されたコンポーネントを上書きできます: |
There was a problem hiding this comment.
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` オプションを使用して、グローバルまたはローカルに登録されたコンポーネントを上書きできます
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
あ、確かにそうですね。そのように修正します!
|
||
mount(Component, { | ||
// 空スタブによってグローバルに登録されたコンポーネントを | ||
// 解決します |
There was a problem hiding this comment.
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 を空のスタブとして解決します。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
確かにそうですね。そのように修正します!
@tell-k |
docs/ja/api/options.md
Outdated
``` | ||
|
||
::: 注意 | ||
`propsData` は Vue Test Utils のマウティングオプションではなく [Vue API](https://vuejs.org/v2/api/#propsData) です。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
誤字 👀
マウティング → マウンティング
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ありがとうございます!
done |
あ、自分権限ないからEdd氏にお願いするという感じでしたね。。。 |
Do you want me to merge @kazupon ? |
@eddyerburgh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for translating :)
No description provided.