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

i18n use monent relative time concept #1

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Pliman
Copy link

@Pliman Pliman commented Apr 24, 2014

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.

i18n use monent relative time concept
@Pliman
Copy link
Author

Pliman commented Apr 24, 2014

hm... I should change test as well, later.

@Pliman
Copy link
Author

Pliman commented Apr 24, 2014

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.

@jbrockmeyer
Copy link

Hey, have you checked if "german (de)" is working for the i18n?

jbrockmeyer and others added 2 commits June 16, 2014 17:22
@cmalard
Copy link

cmalard commented Jan 8, 2018

Hi @Pliman and @codebox , happy new year :-)

I would have been interested by this PR, some years have passed, what's the status ? :-)

@Pliman
Copy link
Author

Pliman commented Jan 8, 2018

Hi, @cmalard , the modified files conflicts with current version.

Conflicting files
readable-range-test.js
readable-range.js

Should fix the conficts based on current version, sometime.

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.

None yet

3 participants