Skip to content

Translate: Migration Guide > Data Option #147

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
Nov 3, 2020
Merged

Translate: Migration Guide > Data Option #147

merged 4 commits into from
Nov 3, 2020

Conversation

nalpan
Copy link
Collaborator

@nalpan nalpan commented Oct 26, 2020

@netlify
Copy link

netlify bot commented Oct 26, 2020

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

Built with commit cf116b4

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

@nalpan
Copy link
Collaborator Author

nalpan commented Oct 26, 2020

遅くなりました、よろしくお願いします。

@potato4d potato4d self-requested a review October 27, 2020 06:35
Copy link
Member

@potato4d potato4d left a comment

Choose a reason for hiding this comment

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

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

一点、コメントしました!

@@ -38,13 +37,13 @@ For example:
</script>
```

Though this provided some convenience in terms of root instances having a shared state, this has led to confusion due to the fact that its only possible on the root instance.
これは共有状態を持っているルートインスタンスに関してはある程度の利便性をもたらしましたが、ルートインスタンスでのみ可能であるという事実のため混乱を招きました。
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
これは共有状態を持っているルートインスタンスに関してはある程度の利便性をもたらしましたが、ルートインスタンスでのみ可能であるという事実のため混乱を招きました。
これは共有状態を持っているルートインスタンスに関してはある程度の利便性をもたらしましたが、ルートインスタンスでのみ可能であるという事実のため混乱を招きました。

この shared の edstate にかかっていて、 状態が共有されている と表現できる認識です。いかがでしょう?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

9f89571
にて修正しました!

badges:
- breaking
---

# {{ $frontmatter.title }} <MigrationBadges :badges="$frontmatter.badges" />
# Data オプション <MigrationBadges :badges="$frontmatter.badges" />
Copy link
Member

Choose a reason for hiding this comment

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

こちらなんですが、 frontmatter 側を変更いただきたいですー。

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

e8cebe5
にて修正しました!

@nalpan
Copy link
Collaborator Author

nalpan commented Nov 1, 2020

@potato4d
レビューありがとうございます、各コメント対応しましたので確認お願いします。

Copy link
Member

@potato4d potato4d left a comment

Choose a reason for hiding this comment

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

LGTM!
ありがとうございますー!

@potato4d potato4d merged commit ef9d733 into vuejs-jp:lang-ja Nov 3, 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.

Migration Guide > Data Option の翻訳
2 participants