-
Notifications
You must be signed in to change notification settings - Fork 11.3k
[IMP] javascript reference: add min_display_digits option #16201
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
Conversation
odoo/enterprise#104728 added a `min_display_digits` option on float fields. This commit adds it also in the documentation.
f492a8a to
1be2336
Compare
aab-odoo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Arnaud-Sibille
|
Hello @Feyensv If it is possible for you, it would be great if we can merge this today, as there are a bunch of tickets linked to the fix in the community branch Pinging you as I saw ANV is off (and sorry for the ping) |
Feyensv
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
robodoo r+
8c199f7 introduced minimum precision for float fields, but it has a display issue. Steps to reproduce: - create an invoice, set the price of an an invoice line to `9417.2` -> the web client displays `'9,417.200000000001'` Issue: In case of a minimum precision, the precision was set to 12. But it should be dynamic in regards to the number of integer digits. Solution: The precision is set to 6 digits, but if the number of significant decimal digits is lower, then it is set to that number. The number of significant decimal digits is computed using: 15 - number of integer digits. -- Also implementing the same logic in qweb. To reproduce: - create a an invoice with an invoice line of `300000000031.3` - post and print -> the price unit on the pdf shows "300,000,000,031.300232" opw-5880951 closes #246347 Related: odoo/documentation#16201 Signed-off-by: William André (wan) <[email protected]>
odoo/enterprise#104728 added a `min_display_digits` option on float fields. This commit adds it also in the documentation. closes #16201 Related: odoo/odoo#246347 Signed-off-by: Victor Feyens (vfe) <[email protected]>

https://github.com/odoo/enterprise/pull/104728 added a
min_display_digitsoption on float fields.This commit adds it also in the documentation.