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

[locale] ko: Update ordinal format #3839

Closed
wants to merge 5 commits into from

Conversation

FourwingsY
Copy link
Contributor

almost copied from zh-cn.js

@jsf-clabot
Copy link

jsf-clabot commented Mar 15, 2017

CLA assistant check
All committers have signed the CLA.

locale/ko.js Outdated
@@ -49,8 +49,22 @@ var ko = moment.defineLocale('ko', {
y : '일 년',
yy : '%d년'
},
ordinalParse : /\d{1,2}일/,
Copy link
Contributor

Choose a reason for hiding this comment

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

Please rebase from the HEAD of the develop branch. This was renamed dayOfMonthOrdinalParse 5 days ago.

almost copied from zh-cn.js
@FourwingsY
Copy link
Contributor Author

@fbonzon sorry for late check. i've updated it.

@ichernev
Copy link
Contributor

ichernev commented Aug 3, 2017

@FourwingsY you need to update src/locale/ko.js and src/test/locale/ko.js.

@ichernev ichernev changed the title update ordinal format in korean locale [locale] ko: Update ordinal format Aug 3, 2017
@FourwingsY
Copy link
Contributor Author

@ichernev done.

@ichernev ichernev removed the rewrite label Aug 6, 2017
@ichernev ichernev added this to the 2.19.0 milestone Aug 6, 2017
@ichernev
Copy link
Contributor

ichernev commented Aug 6, 2017

@kyungw00k what do you think?

@ichernev
Copy link
Contributor

ichernev commented Aug 6, 2017

@FourwingsY can you also undo the changes to locale/ko.js, its autogenerated on release.

@kyungw00k
Copy link

@ichernev 👌
@FourwingsY 👍

@FourwingsY
Copy link
Contributor Author

yup

@ichernev ichernev removed the rewrite label Aug 9, 2017
@ichernev
Copy link
Contributor

ichernev commented Aug 9, 2017

Merged in eef541e

@ichernev ichernev closed this Aug 9, 2017
ichernev added a commit that referenced this pull request Aug 9, 2017
[locale] ko: Update ordinal format
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.

5 participants