-
-
Notifications
You must be signed in to change notification settings - Fork 304
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
[FIX] mis_builder: Comparison percentages export format in XLSX #300
[FIX] mis_builder: Comparison percentages export format in XLSX #300
Conversation
Hi @sbidoul, |
c32d65c
to
51fe61f
Compare
When a 'divider' style param is used on a kpi, it is also applied to the variation columns when exporting to Excel, which leads to errors of magnitude for the percentages displayed in the resulting XLSX file.
51fe61f
to
0c1795a
Compare
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.
LGTM, thanks!
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.
I think it should be OK too 👍
@sbidoul merge? |
/ocabot merge |
Hi @sbidoul. Your command failed:
Ocabot commands
More information
|
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at fc36a7c. Thanks a lot for contributing to OCA. ❤️ |
When a divider/factor Style parameter is used on a KPI, it is also applied to the "Compare columns" when exporting to Excel, which leads to errors of magnitude for the percentages displayed in the resulting XLSX file.
Here is a simple report using a KPI with a style factor "1e3 - k" and with a compare column as it is displayed in Odoo :
And the resulting XLSX export:
Seen on 12.0 but it seems like 10.0 code is similar.