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

Added Azerbaijani lang #1628

Merged
merged 1 commit into from May 21, 2014

Conversation

Projects
None yet
2 participants
@topchiyev
Contributor

topchiyev commented Apr 27, 2014

Please accept the pull request for the community.
Thanks.

@ichernev

This comment has been minimized.

Contributor

ichernev commented Apr 30, 2014

Isn't there a nicer way to do the prefixes? There should be plural rules for your language that are somewhat better defined.

@topchiyev

This comment has been minimized.

Contributor

topchiyev commented Apr 30, 2014

Hi ichernev,

Did You mean suffixes? We don't have prefixes for numerals.
I've taken /lang/tr.js as an example as Turkish is the closest relative of Azerbaijani. I have changed some parts of the code to reflect Azerbaijani plural rules. The part with suffixes have also been taken from /lang/tr.js.

Thanks.

@ichernev

This comment has been minimized.

Contributor

ichernev commented May 13, 2014

Hm, I wasn't aware of the tr implementation. The code of ordinal just seems a bit random.

 var a = number % 10,
     b = number % 100 - a,
     c = number >= 100 ? 100 : null;

This part and the part with the suffixes array that only has a few suffixes. What are the actual rules. If you think the implementation makes sense for an Azerbaijani speaker then fine.

@ichernev

This comment has been minimized.

Contributor

ichernev commented May 21, 2014

Oh, well I guess its fine. Thank you for contributing!

ichernev added a commit that referenced this pull request May 21, 2014

@ichernev ichernev merged commit 211078d into moment:develop May 21, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details

@theurere theurere referenced this pull request Jul 1, 2014

Merged

Update moment.js to 2.7.0 #58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment