Update docs in Korean (2.6.0+) #1625

ChangJoo-Park · 2017-07-21T12:24:38Z

Update all docs in Korean :)

kazupon

Thank you for your PR!
Could you review to someone please?

ChangJoo-Park · 2017-07-22T08:39:25Z

I do not have reviewer in Korean officially. I will be looking for a Korean review as soon as possible.
😢

ChangJoo-Park · 2017-07-24T04:08:48Z

@kazupon Please add @kciter to reviewer. He is co-organizer of Vue.js Korean user group.

posva · 2017-07-24T10:59:11Z

I cannot request a review from him in the github menu 😞
He should be able to de the review anyway

kciter · 2017-07-24T11:03:39Z

@posva Okay, I will reivew without reviewer. :)

posva · 2017-07-24T12:27:48Z

@LinusBorg Sorry for the review request xD
@kciter But until now, anybody could review any pr, have that changed 😢 ?

LinusBorg · 2017-07-24T12:31:21Z

@posva I think you are mistaken. You could assign any contributor as a reviewer, not just and "random" github user. kciter isn't a contributor yet.

I thinkk I actually saw requests on twitter to @github to allow multiple times. ..

posva · 2017-07-24T12:36:02Z

@LinusBorg aaah, I didn't know it worked like that. Thanks for the info!

kciter

몇 가지 수정하면 좋을 것 같은 곳에 코멘트를 남겼습니다. :)

kciter · 2017-07-25T01:32:07Z

docs/kr/advanced/data-fetching.md

가져 오는 -> 가져오는
가져 오기 -> 가져오기
이 부분의 맞춤법만 수정하면 될 것 같습니다 :)

감사합니다 👍

kciter · 2017-07-25T01:49:40Z

docs/kr/advanced/lazy-loading.md

-const Baz = r => require.ensure([], () => r(require('./Baz.vue')), 'group-foo')
+const Foo = () => import(/* webpackChunkName: "group-foo" */ './Foo.vue')
+const Bar = () => import(/* webpackChunkName: "group-foo" */ './Bar.vue')
+const Baz = () => import(/* webpackChunkName: "group-foo" */ './Baz.vue')
 ```

 Webpack은 동일한 묶음 이름을 가진 비동기 모듈을 같은 비동기 묶음으로 그룹화합니다. 이것은 또한`require.ensure`에 대한 종속성을 더 이상 명시 적으로 나열할 필요가 없음을 의미합니다(따라서 빈 배열을 전달합니다).


이 부분의 내용이 최신 버전에서는
webpack will group any async module with the same chunk name into the same async chunk.
으로 수정되었습니다.
Webpack은 동일한 이름의 묶음을 가진 비동기 모듈을 동일한 비동기 묶음으로 그룹화합니다.
으로 번역하면 될 것 같습니다.

감사합니다 👍

kciter · 2017-07-25T01:56:06Z

docs/kr/api/options.md

+  > 2.6.0+
+
+  - type: `boolean`


이곳에만 자료형이라고 안적혔습니다.

Translate type to 자료형

kciter · 2017-07-25T01:56:55Z

docs/kr/api/options.md

+   브라우저가 `history.pushState`를 지원하지 않을 때 라우터가 `hash`모드로 변경되어야할지 설정합니다. 기본값은 `true`입니다.
+
+   이를 `false`로 설정하면 IE9에서 모든 라우터 링크를 탐색 할 수 있습니다. 이것은 해시모드 URL이 SSR에서 작동하지 않기 때문에 앱이 서버에서 렌더링되어 IE9에서 작동해야하는 경우에 유용합니다.


라우터 링크 -> router-link가 좋을 것 같습니다. :)

변경했습니다 👍

yu1222 · 2017-07-25T02:28:57Z

docs/kr/advanced/data-fetching.md

@@ -54,7 +54,7 @@ export default {
    fetchData () {
      this.error = this.post = null
      this.loading = true
-      // getPost를 데이터 가져 오기 위한 유틸리티/API 래퍼로 변경합니다.
+      // `getPost`를 데이터 가져 오기 위한 유틸리티/API 래퍼로 변경합니다.


마찬가지로
가져 오기 -> 가져오기
맞춤법 수정이 필요합니다.

감사합니다 👍

yu1222 · 2017-07-25T02:29:51Z

docs/kr/advanced/data-fetching.md

@@ -109,4 +109,4 @@ export default {
 }


98라인에도 57라인과 통일해서 백틱을 넣으면 좋겠습니다.

추가했습니다 👍

yu1222 · 2017-07-25T02:32:21Z

docs/kr/advanced/navigation-guards.md

-8. 전역 `beforeResolve` 가드 호출 (2.5이상)
+1. 네비게이션이 트리거됨.
+2. 비활성화될 컴포넌트에서 가드를 호출.
+3. 전역  `beforeEach` 가드 호출.


(넘나 미미한데 눈이 띄어서......)
'전역' 다음에 공백이 2칸입니다.

놓친 부분인데 감사합니다

yu1222 · 2017-07-25T02:36:40Z

docs/kr/api/options.md

+  - type: `boolean`
+
+   브라우저가 `history.pushState`를 지원하지 않을 때 라우터가 `hash`모드로 변경되어야할지 설정합니다. 기본값은 `true`입니다.


변경되어야할지 -> 변경되어야 할지

감사합니다 👍

yu1222 · 2017-07-25T02:37:16Z

docs/kr/api/router-link.md

@@ -99,7 +99,7 @@
  이것의 결과는 `<router-link to="/">`가 모든 라우터에 대해 활성화 될 것입니다! 링크를 "완전 일치 모드"로 강제하려면 `exact` prop를 사용하십시오.

  ``` html
-  <!-- 이 링크는 / 에서만 active 됩니다 -->
+  <!-- 이 링크는 `/` 에서만 active 됩니다 -->


active -> 활성화

yu1222 · 2017-07-25T02:38:03Z

docs/kr/essentials/named-routes.md

@@ -14,7 +14,7 @@ const router = new VueRouter({
 })
 ```

-명명 된 라우트에 링크하려면, 객체를 `router-link`, 컴포넌트의 `to` prop로 전달할 수 있습니다.
+이름을 가진 된 라우트에 링크하려면, 객체를 `router-link`, 컴포넌트의 `to` prop로 전달할 수 있습니다.


이름을 가진 된 -> 이름을 가진

감사합니다 👍

yu1222 · 2017-07-25T02:38:29Z

docs/kr/essentials/named-routes.md

@@ -14,7 +14,7 @@ const router = new VueRouter({
 })
 ```

-명명 된 라우트에 링크하려면, 객체를 `router-link`, 컴포넌트의 `to` prop로 전달할 수 있습니다.
+이름을 가진 된 라우트에 링크하려면, 객체를 `router-link`, 컴포넌트의 `to` prop로 전달할 수 있습니다.


prop로 -> prop으로
(props 가 아니기 때문에.. "으로"가 더 잘 읽히네요)

네 저도 다시 읽어보니 맞는 것 같아 말씀해주신대로 변경했습니다

yu1222 · 2017-07-25T02:41:08Z

docs/kr/advanced/lazy-loading.md

+ 이 두 가지를 결합하여 Webpack에 의해 자동으로 코드 분할될 비동기 컴포넌트를 정의하는 방법입니다.
+
+ ``` js
+ const Foo = () => import('./Foo.vue')
 ```

 라우트 설정에서 아무것도 바꿀 필요가 없습니다. 보통 `Foo`만 사용하십시오.


보통 Foo만 사용하십시오.
→ 보통 Foo만 사용하면 됩니다. (?)

보통을 빼고 Foo만 사용하면 됩니다. 로 변경했습니다.

ChangJoo-Park · 2017-07-25T10:44:06Z

@kciter @yu1222 말씀해주신 내용에 맞춰 수정했습니다. 정말 감사합니다.

@kazupon Now, I've got awesome reviewers! 😄
Please check commits and reviews.

posva · 2017-07-25T11:58:46Z

@ChangJoo-Park ping me when it's ready 🙂
Thanks a lot 👍

kazupon · 2017-07-25T12:01:17Z

@ChangJoo-Park Thanks! Also for reviewers!

ChangJoo-Park added 16 commits July 21, 2017 20:41

Update SUMMARY.md

1ef6605

Update navigation-guards

af1041e

Update getting-started

edbf51f

Update history-mode

6055f7d

Update named-routes

c8dc391

Rollback example url

c6a3c22

Update passing-props.md

2a0586e

Update passing-props

5d0ed39

Update data-fetching

70254cc

Update lazy-loading.md

afd9ad7

Update navigation-guards

6bd915b

Update scroll-behavior.md

0c8d341

Update transitions.md

bc0b5c9

Update options.md

c7d3933

Update router-instance.md

007c12c

Update router-link.md

3073cea

kazupon reviewed Jul 21, 2017

View reviewed changes

posva requested a review from LinusBorg July 24, 2017 10:58

posva removed the request for review from LinusBorg July 24, 2017 12:27

kciter reviewed Jul 25, 2017

View reviewed changes

yu1222 reviewed Jul 25, 2017

View reviewed changes

ChangJoo-Park added 2 commits July 25, 2017 19:38

fix by Review - data-fetching

c3d4b04

fix by Review - lazy-loading

fed7c91

ChangJoo-Park added 4 commits July 25, 2017 19:38

fix by Review - navigation-guards

6275c16

fix by Review - API

6118e8d

fix by Review - named routes

c99b528

Fix by review - options router-link

eaea2b0

kazupon merged commit 7f33d1e into vuejs:dev Jul 25, 2017

		브라우저가 `history.pushState`를 지원하지 않을 때 라우터가 `hash`모드로 변경되어야할지 설정합니다. 기본값은 `true`입니다.

		이를 `false`로 설정하면 IE9에서 모든 라우터 링크를 탐색 할 수 있습니다. 이것은 해시모드 URL이 SSR에서 작동하지 않기 때문에 앱이 서버에서 렌더링되어 IE9에서 작동해야하는 경우에 유용합니다.

		- type: `boolean`

		브라우저가 `history.pushState`를 지원하지 않을 때 라우터가 `hash`모드로 변경되어야할지 설정합니다. 기본값은 `true`입니다.

Uh oh!

Update docs in Korean (2.6.0+) #1625

Update docs in Korean (2.6.0+) #1625

Uh oh!

Conversation

ChangJoo-Park commented Jul 21, 2017

Uh oh!

kazupon left a comment

Choose a reason for hiding this comment

Uh oh!

ChangJoo-Park commented Jul 22, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ChangJoo-Park commented Jul 24, 2017

Uh oh!

posva commented Jul 24, 2017

Uh oh!

kciter commented Jul 24, 2017

Uh oh!

posva commented Jul 24, 2017

Uh oh!

LinusBorg commented Jul 24, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

posva commented Jul 24, 2017

Uh oh!

kciter left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ChangJoo-Park commented Jul 25, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

posva commented Jul 25, 2017

Uh oh!

ChangJoo-Park commented Jul 22, 2017 •

edited

Loading

LinusBorg commented Jul 24, 2017 •

edited

Loading

ChangJoo-Park commented Jul 25, 2017 •

edited

Loading