jsgoupil/quickbooks-sync

decimal precision in QUANTYPE

Davidcc opened this issue · 6 comments

Hello, we are trying to send quantities such as 0.16667 as the quantity on a CreditMemoLineAdd, and the QBXML being generated is 0.17.

I believe this is due to the use of
return _value.ToString("F", CultureInfo.InvariantCulture);

in the FLOATTYPE superclass. Since (I believe) QBXML supports 5 decimal places, would "F5" be more appropriate here?

Sounds right. You could send a PR with unit tests, I'll be happy to merge.
FYI, AmtType can only have 2 decimals

I have finished the changes but I think I need someone with more permissions to create a branch?

You should create a fork, commit to your fork, then in github it will ask you to do a pull request.

Thanks for your help!

No problem, I'll publish this today

This is released as 3.0.2
I have released the WebConnector as well with the new version since we changed a public API (private->protected)