-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
time parse error #1093
Comments
(I am not a maintainer here, but): did you try running from the top of the tree, i.e. from master? Because other time parsing bugs have been fixed since the most recent release. I recently hit #1080 which isn't that different. |
my excelize version is v2.4.2 |
@xuri @danielbprice Have you resolved this problem? I met same after: |
This issue has been fixed. Could you provide spreadsheet attachment and standalone code to reproduce this issue? |
ENV: require ( attachments: test.xlsx
result: '12-17-21 12-18-21 12-19-21 12-20-21 12-21-21 12-22-21 |
The file attachments upload failed. |
@xuri uploaded. |
This patch fixed the accuracy issue on parsing cells with time number format in some cases, but not related to language location. The library support parsing build-in date and time number format under English (United States) currently, so the cell value should be concat with a dash (not slash) after applying the number format, I'll consider adding language location support later. |
…is improve number format support - Introduced NFP (number format parser) dependencies module - Initialize custom dates and times number format support - Dependencies module upgraded
…is improve number format support - Introduced NFP (number format parser) dependencies module - Initialize custom dates and times number format support - Dependencies module upgraded
the excel data:
after parse, the output is:
the code is:
The text was updated successfully, but these errors were encountered: