search

jsgoupil / quickbooks-sync

Sync Quickbooks Desktop
MIT License
89 stars 40 forks source link

support 5 decimal places in QUANTYPE and 2 for AMTYPE #57

Closed Davidcc closed 3 years ago

jsgoupil commented 3 years ago

Thanks. I'll apply some formatting afterwards. But can you add tests for a float that's not with decimal data, also, all other types are supposed to be with 5 decimals as well (like rate).

I'm on vacation until Tuesday 😁

jsgoupil commented 3 years ago

I can add those tests if you don't have time. Also we should have tests to see if it's a away from zero or banking rounding.

jsgoupil commented 3 years ago

Close in favor of #58