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

Bug: Parsing AM/PM always returns AM #35

Closed
bagrounds opened this issue Dec 16, 2017 · 1 comment
Closed

Bug: Parsing AM/PM always returns AM #35

bagrounds opened this issue Dec 16, 2017 · 1 comment

Comments

@bagrounds
Copy link

const time_am = '2017-12-15 2:00AM'
const time_pm = '2017-12-15 2:00PM'
const format = 'YYYY-MM-dd h:mma' // 'a' is for AM/PM
tc.DateTime.parse(time_am, format).format(format)
// => '2017-12-15 2:00AM'
tc.DateTime.parse(time_pm, format).format(format)
// => '2017-12-15 2:00AM' (<- this should be PM)
$ node --version
v8.9.3
$ npm ls timezonecomplete
└── timezonecomplete@5.4.5
rogierschouten pushed a commit that referenced this issue Dec 16, 2017
@rogierschouten
Copy link
Contributor

Thank you for reporting this. It's fixed and published as v5.4.6.

Also I documented our level of support for various LDML patterns

As you can see, there is still a bit of work to be done there. If you need anything else that is not supported, please let me know.

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

No branches or pull requests

2 participants