search

xavier / xlsx_reader

A production-ready XLSX file reader for Elixir.
105 stars 17 forks source link

fix: give preference for dates over numeric types #33

Closed ding-an-sich closed 5 months ago

ding-an-sich commented 5 months ago

elixslx gives type numeric to cells with excel timestamps.

This PR adds two matches to give preference for date/datetime parsing over numeric parsing.

I'm not sure this follows the specs, but it seems safe enough to support.

If you prefer to file a bug for elixslx let me know.

xavier commented 5 months ago

It's not the first compatibility hack and won't be the last. 😉

Thanks for your contribution! 👍