fix for issue #1332: Incorrect parsing of dates if crossing DST boundary (with fixed unit tests) #1343

Closed
lemmingworks wants to merge 0 commits from master into master
lemmingworks commented 2018-11-06 15:02:33 +00:00 (Migrated from github.com)

This PR is a clone of PR 1333, with fixes for the failing unit tests.

In my use case, using this package to parse dates that were in British Summer Time (GMT+1) results in the previous day now that we are in GMT again, as the differences in timezone are not taken into account.

This is a fix for Issue 1332

This PR is a clone of [PR 1333](https://github.com/SheetJS/js-xlsx/pull/1333), with fixes for the failing unit tests. In my use case, using this package to parse dates that were in British Summer Time (GMT+1) results in the previous day now that we are in GMT again, as the differences in timezone are not taken into account. This is a fix for [Issue 1332](https://github.com/SheetJS/js-xlsx/issues/1332)
lemmingworks commented 2018-11-07 15:40:27 +00:00 (Migrated from github.com)

Closing because @lalomartins has updated his pull request to include the unit test fixes. So, as our pull requests are now equivalent, I'm deleting this one to avoid confusion.

Closing because @lalomartins has updated his pull request to include the unit test fixes. So, as our pull requests are now equivalent, I'm deleting this one to avoid confusion.

Pull request closed

Sign in to join this conversation.
No description provided.