-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Translations for Dates (fixes #1584) #1605
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
Conversation
- new camel cased imports - unicode date formatter codes - renamed parse to parseISO
|
Hi @andrewn Trying to test the |
Good spot! I've pushed a new commit that removes this warning. |
|
Great, indeed, the translations for dates work. |
oruburos
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, indeed, the translations for dates work. Timer messages in the bar are not displayed but I checked against messages in console and they were correct. So functionality for this PR is there.
I tested this and I'm also not seeing the timer messages! |
I couldn't see the Timer, but I think that's unrelated to this PR as I don't see it in the develop branch either. Or on the deployed editor. Do you know what the behaviour of the Timer is and I'll try and fix it....! |
Oh thanks for pointing that out! It's the Timer.jsx component that renders in the Editor.jsx component. Since this is unrelated to this PR, I'll open a new issue so it can be merged in a separate PR :) |
|
Just fixed the Timer issue with #1617 :) |
catarak
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
okay this is now working as expected for me 😄 so rad!
dates.format(),dates.distanceInWordsToNow())Both locales are imported into the bundle at the moment. They're only 1.5k each so not a huge issue but as we add more languages we might want to dynamically import each locale as needed. This will take more work as the components expect the formatting to be synchronous at the moment.
Fixes #1584
I have verified that this pull request:
npm run lint)Fixes #1584