-
Notifications
You must be signed in to change notification settings - Fork 91
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
i18n use monent relative time concept #1
base: master
Are you sure you want to change the base?
Conversation
i18n use monent relative time concept
hm... I should change test as well, later. |
retrieve test
retrieved tests, ‘1 second’ and'xx seconds' now will be 'a few seconds' which comes from moment i18n file, I think this is not perfect. |
Hey, have you checked if "german (de)" is working for the i18n? |
…n-concept change to have i18n with relative time from moment
Hi, @cmalard , the modified files conflicts with current version. Conflicting files Should fix the conficts based on current version, sometime. |
Hello, it nice to find moment-precise-range, but it doesn't support i18n, so I added i18n support with monent relative time concept.
One thing not good is that, now 'a' will replace '1', eg. '1 month' -> 'a month'
but moment did like this, make be it will change in the future, probably moment-precise-range with i18n support can still work.