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

fix: update ja docs #3214

Merged
merged 20 commits into from
Jun 7, 2020
Merged

fix: update ja docs #3214

merged 20 commits into from
Jun 7, 2020

Conversation

kazupon
Copy link
Member

@kazupon kazupon commented May 29, 2020

No description provided.

@kazupon
Copy link
Member Author

kazupon commented May 29, 2020

@kiaking @ktsn
I've jsut updated ja docs after long time.
please review this PR!

docs/ja/api/README.md Outdated Show resolved Hide resolved
@@ -34,13 +34,13 @@ router.beforeEach((to, from, next) => {

- **`next(error)`**: (2.4.0+) `next` に渡された引数が `Error` インスタンスである場合、ナビゲーションは中止され、エラーは `router.onError()` を介して登録されたコールバックに渡されます。

**Make sure that the `next` function is called exactly once in any given navigation guard. It can appear more than once, but only if the logical paths have no overlap, otherwise the hook will never be resolved or produce errors.** Here is an example of redirecting to user to `/login` if they are not authenticated:
**与えれれた任意のナビゲーションガードで、常に 1回だけ `next` 関数が呼び出されるようにしてください。それは複数回呼び出すことができますが、論理パスが重ならないときだけで、そうしないないとフックは決して解決されない、またはエラーが発生します。** 以下は、ユーザーが認証されていない場合、`/login` にリダレクトするための例です:
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
**与えれれた任意のナビゲーションガードで、常に 1回だけ `next` 関数が呼び出されるようにしてください。それは複数回呼び出すことができますが、論理パスが重ならないときだけで、そうしないないとフックは決して解決されない、またはエラーが発生します。** 以下は、ユーザーが認証されていない場合、`/login` にリダレクトするための例です:
**与えられた任意のナビゲーションガードで、常に 1回だけ `next` 関数が呼び出されるようにしてください。それは複数回呼び出すことができますが、論理パスが重ならないときだけで、そうしないとフックは決して解決されない、またはエラーが発生します。** 以下は、ユーザーが認証されていない場合、`/login` にリダレクトするための例です:

@@ -34,13 +34,13 @@ router.beforeEach((to, from, next) => {

- **`next(error)`**: (2.4.0+) `next` に渡された引数が `Error` インスタンスである場合、ナビゲーションは中止され、エラーは `router.onError()` を介して登録されたコールバックに渡されます。

**Make sure that the `next` function is called exactly once in any given navigation guard. It can appear more than once, but only if the logical paths have no overlap, otherwise the hook will never be resolved or produce errors.** Here is an example of redirecting to user to `/login` if they are not authenticated:
**与えれれた任意のナビゲーションガードで、常に 1回だけ `next` 関数が呼び出されるようにしてください。それは複数回呼び出すことができますが、論理パスが重ならないときだけで、そうしないないとフックは決して解決されない、またはエラーが発生します。** 以下は、ユーザーが認証されていない場合、`/login` にリダレクトするための例です:
Copy link
Member

Choose a reason for hiding this comment

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

それは複数回呼び出すことができますが

The original sentence tells "It can appear more than once" which means you can write next function more than once in your code. You still can't call it twice.

Copy link
Member Author

Choose a reason for hiding this comment

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

yeah, you're right ... 😅

docs/ja/installation.md Outdated Show resolved Hide resolved
kazupon and others added 3 commits June 7, 2020 18:59
Co-authored-by: Katashin <ktsn55@gmail.com>
Co-authored-by: Katashin <ktsn55@gmail.com>
@kazupon
Copy link
Member Author

kazupon commented Jun 7, 2020

@ktsn
Thank you for your reviewing!
I've just fixed.

Copy link
Member

@ktsn ktsn left a comment

Choose a reason for hiding this comment

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

LGTM

@kazupon kazupon merged commit c05f741 into vuejs:dev Jun 7, 2020
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.

2 participants