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

Add Korean locale #206

Merged
merged 2 commits into from
Jun 2, 2018
Merged

Add Korean locale #206

merged 2 commits into from
Jun 2, 2018

Conversation

ujuc
Copy link
Contributor

@ujuc ujuc commented May 29, 2018

Korean locale!!

#171

@ujuc
Copy link
Contributor Author

ujuc commented May 29, 2018

awaiting merge this issue.

I would like to ask my friends if the translation is correct.

@codecov-io
Copy link

codecov-io commented May 29, 2018

Codecov Report

Merging #206 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #206   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          18     19    +1     
  Lines         322    325    +3     
  Branches       50     50           
=====================================
+ Hits          322    325    +3
Impacted Files Coverage Δ
src/locale/ko.js 100% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c37b3da...c0dd5b7. Read the comment docs.

@iamkun
Copy link
Owner

iamkun commented May 29, 2018

@ujuc Great work thanks.

Beside, fell free to add a link to your work in our main readme file like other translations. 😁


### Validation `.isValid()`

`Dayjs` 날짜가 유효한지 확인하도록 `boolean` 값으로 반환합니다.
Copy link

@arzhna arzhna May 29, 2018

Choose a reason for hiding this comment

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

Dayjs 날짜가 유효한지 확인합니다. 반환 형식은 boolean 입니다.

정도로 고치면 좋겠어요.


Day.js는 네이티브 `Date.prototype`을 수정하는 대신 `Dayjs` 오브젝트인 Date 오브젝트 래퍼를 생성합니다.

`Dayjs` 오브젝트는 변경이 불가능(immutable)합니다. 즉, 모든 API 작업은 새로운 `Dayjs` 객체를 반환해야 합니다.
Copy link

Choose a reason for hiding this comment

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

Dayjs 오브젝트는 변경할 수 없습니다(immutable). 즉, 'Dayjs' 오브젝트를 변경하는 모든 API 작업은 새로운 Dayjs 오브젝트를 반환합니다.

용어(오브젝트객체)를 혼용하지 말고 하나로 통일하는게 좋겠죠? 그리고 원문은 will return이니 반환해야 합니다가 아닌 반환합니다가 적합할 것 같네요.


### Day of the Month `.date()`

`Dayjs`에서 일자를 `number` 타입 값으로 반환합니다.
Copy link

@arzhna arzhna May 29, 2018

Choose a reason for hiding this comment

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

아래 내용을 보니 일자와 날짜가 혼용되어 있네요. 일자(日子) 보다는 날짜로 통일하는게 어떨까요?


### Day of the Week `.day()`

`Dayjs`에서 주에 대한 일자를 `number` 타입 값으로 반환합니다.
Copy link

Choose a reason for hiding this comment

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

주에 대한 일자는 말이 좀 어렵네요. 요일로 바꾸면 깔끔해 질 것 같습니다.


### Set `.set(unit: string, value: number)`

적용된 번경사항이 있는 `Dayjs`를 반환합니다.
Copy link

@arzhna arzhna May 30, 2018

Choose a reason for hiding this comment

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

변경 사항이 적용된 Dayjs를 반환합니다.

정도로 고치면 좋겠어요.


### Add `.add(value: number, unit: string)`

특정 시간이 추가하여 복제된 `Dayjs`를 반환합니다.
Copy link

@arzhna arzhna May 30, 2018

Choose a reason for hiding this comment

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

지정한 시간을 더한 Dayjs의 복제본을 반환합니다.


### Subtract `.subtract(value: number, unit: string)`

특정 시간이 감소하여 복제된 `Dayjs`를 반환합니다.
Copy link

@arzhna arzhna May 30, 2018

Choose a reason for hiding this comment

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

지정한 시간을 뺀 Dayjs의 복제본을 반환합니다.


### Format `.format(stringWithTokens: string)`

`Dayjs`시간을 기본 형식으로 `string` 타입 값으로 반환합니다.
Copy link

Choose a reason for hiding this comment

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

Dayjs 시간을 기본 형식의 문자열(string)로 반환합니다.

| `A` | AM PM | |
| `a` | am pm | |

* 플러그인 [`AdvancedFormat`](./Plugin.md#advancedformat) 을 사용하면 더 많은 형식 (`Q Do k kk X x ...`)을 사용할 수 있습니다.
Copy link

Choose a reason for hiding this comment

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

  • 플러그인 AdvancedFormat을 사용하면 더 많은 형식(Q Do k kk X x ...)을 사용할 수 있습니다.

명사(부연설명) + 조사는 붙여써야 합니다.


### Difference `.diff(compared: Dayjs, unit: string (default: 'milliseconds'), float?: boolean)`

특정 시간을 가르키는 두 `Dayjs`에 대한 차이를 `number` 타입 값으로 반환합니다.
Copy link

Choose a reason for hiding this comment

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

두 'dayjs'의 차이를 지정한 단위로 반환합니다. 반환 형식은 number 입니다.


### As JSON `.toJSON()`

ISO8601 `string` 타입 형식으로 `Dayjs`를 반환합니다.
Copy link

Choose a reason for hiding this comment

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

타입 형식은 중복인것 같네요. 하나만 쓰는게 좋겠어요.


### Is Before `.isBefore(compared: Dayjs)`

`Dayjs` 값이 다른 `Dayjs` 값보다 앞선 시점인지를 `boolean` 타입 값으로 반환합니다.
Copy link

Choose a reason for hiding this comment

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

Dayjs가 다른 Dayjs보다 앞선 시점인지를 ...

은 빼도 될 것 같습니다.

src/locale/ko.js Outdated
months: '1월_2월_3월_4월_5월_6월_7월_8월_9월_10월_11월_12월'.split('_'),
ordinal: n => n,
relativeTime: {
future: '%s 아내',
Copy link

Choose a reason for hiding this comment

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

아내? 원문이 in이니 안에 또는 이내 정도가 될 것 같네요.

docs/ko/I18n.md Outdated
@@ -0,0 +1,110 @@
## Internationalization

Day.js는 국제화에 대해 많은 지원을 합니다.
Copy link

Choose a reason for hiding this comment

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

Internationalization을 직역하면 국제화지만 다국어 지원에 대한 내용이니 Day.js는 다국어를 지원합니다. 정도로 번역하는게 좋을 것 같습니다.

docs/ko/I18n.md Outdated

Day.js에서 기본 locale 값은 영어 (미국) 입니다.

여러 locale을 로드하고 싶다면 쉽게 locale간 전환 할 수 있습니다.
Copy link

Choose a reason for hiding this comment

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

원문은 You can load multiple locales and switch between them easily.입니다.
여러 locale을 로드하고 쉽게 전환할 수 있습니다.가 적절할 것 같네요.

@arzhna
Copy link

arzhna commented May 30, 2018

전체적으로 너무 번역기 문체인 것 같아요. 문장들을 매끄럽게 다듬을 필요가 있습니다.

docs/ko/I18n.md Outdated

Day.js는 국제화에 대해 많은 지원을 합니다.

그러나 그것을 사용하지 않는다면, 그 누구도 당신의 빌드에 포함되지 않습니다.
Copy link

Choose a reason for hiding this comment

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

다국어는 당신이 사용할 때만 빌드에 포함됩니다.
문장 만들기 어렵네요 :)

Korean locale!!
Copy link
Contributor Author

@ujuc ujuc left a comment

Choose a reason for hiding this comment

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

@arzhna :) 감사합니다. 수정했습니다.

@iamkun
Copy link
Owner

iamkun commented May 31, 2018

@ujuc Hi, any progress here so far? Should I merge this now?

@ujuc
Copy link
Contributor Author

ujuc commented Jun 1, 2018

OK do merge!!! :)

@iamkun iamkun merged commit 1de6105 into iamkun:master Jun 2, 2018
@iamkun
Copy link
Owner

iamkun commented Jun 6, 2018

🎉 This PR is included in version 1.6.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ujuc ujuc deleted the locale-ko branch November 22, 2018 16:59
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.

4 participants