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

Adding locale and moment computeds #139

Merged
merged 5 commits into from
Jan 23, 2016
Merged

Adding locale and moment computeds #139

merged 5 commits into from
Jan 23, 2016

Conversation

jasonmit
Copy link
Collaborator

@jasonmit jasonmit commented Jan 3, 2016

Fixes #135

This is API breaking due to the duration computed having to be split out into duration + humanize in order to have the ability to localize duration before humanizing through composition.

durationExample: humanize(locale(duration(10, 'hours'), 'fr')) // => 10 heures

@jasonmit jasonmit force-pushed the locale-support branch 9 times, most recently from 477a00d to cd01cea Compare January 3, 2016 09:08
@jasonmit
Copy link
Collaborator Author

jasonmit commented Jan 3, 2016

@stefanpenner need a review when you can. I'm holding off on merging this one until you do since I want to get it right.

@jasonmit
Copy link
Collaborator Author

jasonmit commented Jan 7, 2016

Paging Mr. @stefanpenner

@stefanpenner
Copy link
Collaborator

Paging Mr. @stefanpenner

Mr @stefanpenner!!

@stefanpenner
Copy link
Collaborator

LVGTM 👍

jasonmit added a commit that referenced this pull request Jan 23, 2016
Adding locale and moment computeds
@jasonmit jasonmit merged commit 8c7e702 into master Jan 23, 2016
@jasonmit jasonmit deleted the locale-support branch January 23, 2016 07:01
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

2 participants