Conversation
| } | ||
| Util.inherits(XlsxStreamReaderWorkSheet, Stream) | ||
|
|
||
| const tryFormat = function (format, workingVal) { |
There was a problem hiding this comment.
I tried to refactor a bit, complexity is beginning to pile up in the formatting code
|
Hi @lambrojos , I have same problem as you, tried to fork your repository and install master branch, also show-formats, but it fills formats with undefined or <1 empty item>, right now the library is parsing date as 1904 time format, can I somehow make this work ? Thanks |
|
Hi @krutijan1 , I just submitted a PR for adding support to 1904 date format. |
|
Hi @krutijan1 sorry for the very late answer. I fixed this and other parsing corner cases in there. If you want to take a look code or backport the code: https://github.com/lambrojos/supersalmon/blob/master/lib/worksheet.js |
A customer asked for the possiblity to see date formats, and so I had to customize the library. I'm sending this pr to ask if maybe you want to integrate this feature as well