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

Quotes in formatCode case handled, standard format 22 changed to m/d/yy hh:mm #186

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ash-ishh
Copy link

@ash-ishh ash-ishh commented Oct 14, 2019

Problem 1: While converting date of following excel file format was detected as mmm d", "yyyy and converted to March 14", "2019

excel_date_format_with_quote

formatCode in xml header was <numFmt numFmtId="165" formatCode="MMMM\ D&quot;, &quot;YYYY"/>

Since date does not generally contain quotes hence removed all quotes in formatCode extracted.

Problem 2: Date 09/01/19 13:00 was detected as dd/mm/yy h:mm and converted to CSV as 09/01/19 01:00 from standard format 22
Since date without am/pm is generally in 24 hour format hence changed it to m/d/yy hh:mm

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

Successfully merging this pull request may close these issues.

1 participant