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

yyyy/mm/dd date format #107

Closed
idpaterson opened this issue Jul 17, 2015 · 1 comment · Fixed by #108
Closed

yyyy/mm/dd date format #107

idpaterson opened this issue Jul 17, 2015 · 1 comment · Fixed by #108

Comments

@idpaterson
Copy link
Collaborator

A user in Japan has noticed that his locale's customary yyyy/mm/dd format is not supported. This issue was briefly discussed in #70 with a potential solution at least when the year has 4 digits that would detect the yyyy/mm/dd format based on the position of the 4-digit sequence, regardless of the current locale's preferred date format. I think that's reasonable. Installing ICU didn't seem to help because this was skipped by the RE_DATE regex.

image

@idpaterson
Copy link
Collaborator Author

According to Wikipedia, only Kazakh uses yyyy.dd.mm, all other national date formats beginning with yyyy are in the order yyyy.mm.dd. I realized that ISO 8601 standard format yyyy-mm-dd is not matched either, so I will handle that case too.

It seems like the yyyy/mm/dd format should have worked for this user when ICU was installed, I'm not quite sure why it did not.

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 a pull request may close this issue.

1 participant