ImportJSON icon indicating copy to clipboard operation
ImportJSON copied to clipboard

Parse numbers from string by default and add 'noParseNumbers' option …

Open ralphilius opened this issue 8 years ago • 6 comments

…as recommended in bradjasper/ImportJSON#7

ralphilius avatar Dec 24 '17 14:12 ralphilius

Hi guys, any chance to get this feature soon please ?

adriens avatar Feb 16 '18 07:02 adriens

I've tested the patch and it does perfectly does the job, thanks a lot @ralphilius 👍

adriens avatar Feb 16 '18 08:02 adriens

Please merge this PR so i can switch back to this repo 🙏

adriens avatar Feb 16 '18 08:02 adriens

Works well for my needs, thanks!

rcourtna avatar Sep 19 '18 15:09 rcourtna

Works bad with documents with non-English locale. We use "," decimal delimiter in Russia. When I use your updated script 1295,84335 becomes 129584335.

dandaka avatar Feb 13 '19 12:02 dandaka

@ralphilius Thank you for sharing! I've applied your change and noticed that blank cells become #NUM! error. This can be avoided using Number(data[row][column]) instead of parseFloat(data[row][column]) to do the actual conversion. This works better for me and might be helpful for others as well.

fghber avatar Jan 05 '21 02:01 fghber