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

Moment 1.5 does not parse "YYYYMMDD" formatted dates. #245

Closed
sfsekaran opened this issue Mar 31, 2012 · 5 comments
Closed

Moment 1.5 does not parse "YYYYMMDD" formatted dates. #245

sfsekaran opened this issue Mar 31, 2012 · 5 comments
Assignees
Milestone

Comments

@sfsekaran
Copy link

For example:

moment("20120115", "YYYYMMDD").valueOf() // => NaN
@timrwood
Copy link
Member

timrwood commented Apr 4, 2012

Duplicate of #157, #79, #108.

There are no plans for adding support for this at the moment due to the increased complexity and filesize that it would bring.

Pull requests are welcome though, (as long as the filesize doesn't blow up)

@sfsekaran
Copy link
Author

Fair enough, hopefully I'll have some time soon to help out :)

@timrwood
Copy link
Member

We have figured out a way to add support for this without too much additional code. Check out the progress at https://github.com/timrwood/moment/tree/feature/parser

@sfsekaran
Copy link
Author

Cool, thanks for the update :)

Sathya Sekaran

On Tuesday, April 10, 2012 at 8:16 PM, Tim Wood wrote:

We have figured out a way to add support for this without too much additional code. Check out the progress at https://github.com/timrwood/moment/tree/feature/parser


Reply to this email directly or view it on GitHub:
#245 (comment)

@timrwood
Copy link
Member

This will go out in the 1.6.0 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants